diff options
| author | Roland Dreier <rolandd@cisco.com> | 2005-10-03 16:18:02 (GMT) |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2005-10-17 22:20:27 (GMT) |
| commit | f575394f1de9b4afa4b474f1882c7e2d3b8e51d7 (patch) | |
| tree | b9ffc5983cbba3acbb8d7233d26101c38f4d7e61 /drivers/infiniband/core | |
| parent | 9825051e8c81cbd4400333e7cdc04be77a0bab75 (diff) | |
| download | linux-fsl-qoriq-f575394f1de9b4afa4b474f1882c7e2d3b8e51d7.tar.xz | |
[IB] uverbs: reject invalid memory registration permission flags
Reject userspace memory registrations with invalid permission flags:
"local write" is required if "remote write" or "remote atomic" is also
requested.
Pointed out by Jack Morgenstein <jackm@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/core')
| -rw-r--r-- | drivers/infiniband/core/uverbs_cmd.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/infiniband/core/uverbs_cmd.c b/drivers/infiniband/core/uverbs_cmd.c index a0ceb1c..2e959ac 100644 --- a/drivers/infiniband/core/uverbs_cmd.c +++ b/drivers/infiniband/core/uverbs_cmd.c @@ -396,6 +396,14 @@ ssize_t ib_uverbs_reg_mr(struct ib_uverbs_file *file, if ((cmd.start & ~PAGE_MASK) != (cmd.hca_va & ~PAGE_MASK)) return -EINVAL; + /* + * Local write permission is required if remote write or + * remote atomic permission is also requested. + */ + if (cmd.access_flags & (IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_REMOTE_WRITE) && + !(cmd.access_flags & IB_ACCESS_LOCAL_WRITE)) + return -EINVAL; + obj = kmalloc(sizeof *obj, GFP_KERNEL); if (!obj) return -ENOMEM; |