diff options
author | Jason Gunthorpe <[email protected]> | 2018-06-12 09:40:23 -0600 |
---|---|---|
committer | Jason Gunthorpe <[email protected]> | 2018-06-12 14:39:32 -0600 |
commit | 1eb9364ce81d9445ad6f9d44921a91d2a6597156 (patch) | |
tree | 027b021e6efb08f389e12e1bd7af94bfc7b3f323 /lib/test_overflow.c | |
parent | 3dc7c7badb7502ec3e3aa817a8bdd9e53aa54c52 (diff) |
IB/uverbs: Fix ordering of ucontext check in ib_uverbs_write
During disassociation the ucontext will become NULL, however due to how
the SRCU locking works the ucontext must only be examined after looking
at the ib_dev, which governs the RCU control flow.
With the wrong ordering userspace will see EINVAL instead of EIO for a
disassociated uverbs FD, which breaks rdma-core.
Cc: [email protected]
Fixes: 491d5c6a3023 ("RDMA/uverbs: Move uncontext check before SRCU read lock")
Reported-by: Mark Bloch <[email protected]>
Signed-off-by: Jason Gunthorpe <[email protected]>
Reviewed-by: Leon Romanovsky <[email protected]>
Diffstat (limited to 'lib/test_overflow.c')
0 files changed, 0 insertions, 0 deletions