summaryrefslogtreecommitdiffstats
path: root/include/rdma/ib_mad.h
diff options
context:
space:
mode:
authorJack Morgenstein <jackm@dev.mellanox.co.il>2009-02-17 23:51:47 +0100
committerRoland Dreier <rolandd@cisco.com>2009-02-17 23:51:47 +0100
commit71d98b4628ee869d62814f6d8607d76cab4b9ec5 (patch)
tree10b4d085aaf23020c052d5e340330d757730d138 /include/rdma/ib_mad.h
parentMerge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/rol... (diff)
downloadlinux-71d98b4628ee869d62814f6d8607d76cab4b9ec5.tar.xz
linux-71d98b4628ee869d62814f6d8607d76cab4b9ec5.zip
IPoIB: In unicast_arp_send(), only free newly-created paths
If path_rec_start() returns error, call path_free() only if the path was newly-created. If we free an existing path whose valid flag was zero, (but do not detach it from the list) we cause corruption of the path list (of which it is a member), and get a kernel crash. The simplest solution is to not free an existing path -- just leave it in the list as-is (i.e., with its valid flag cleared). Thanks to Yossi Etigin of Voltaire for identifying the problem flow which caused the kernel crash. Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il> Signed-off-by: Moni Shua <monis@voltaire.com> Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'include/rdma/ib_mad.h')
0 files changed, 0 insertions, 0 deletions