aboutsummaryrefslogtreecommitdiff
path: root/CREDITS
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-10-13 09:13:12 +0000
committerWilly Tarreau <w@1wt.eu>2011-02-09 22:15:36 +0100
commitfca909254747217cbd454e9913fedfc728704828 (patch)
tree1d85835ed743e9a1794b0e323d00d0c6730895ac /CREDITS
parentaa076d3e48c4e19717fd20b840f288ccc8bab839 (diff)
IB/uverbs: Handle large number of entries in poll CQ
commit 7182afea8d1afd432a17c18162cc3fd441d0da93 upstream. In ib_uverbs_poll_cq() code there is a potential integer overflow if userspace passes in a large cmd.ne. The calls to kmalloc() would allocate smaller buffers than intended, leading to memory corruption. There iss also an information leak if resp wasn't all used. Unprivileged userspace may call this function, although only if an RDMA device that uses this function is present. Fix this by copying CQ entries one at a time, which avoids the allocation entirely, and also by moving this copying into a function that makes sure to initialize all memory copied to userspace. Special thanks to Jason Gunthorpe <jgunthorpe@obsidianresearch.com> for his help and advice. Signed-off-by: Dan Carpenter <error27@gmail.com> [ Monkey around with things a bit to avoid bad code generation by gcc when designated initializers are used. - Roland ] Signed-off-by: Roland Dreier <rolandd@cisco.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> Signed-off-by: Willy Tarreau <w@1wt.eu>
Diffstat (limited to 'CREDITS')
0 files changed, 0 insertions, 0 deletions