diff options
| author |   Jack Morgenstein <jackm@dev.mellanox.co.il> | 2009-02-17 14:51:47 -0800 |
|---|---|---|
| committer |   Roland Dreier <rolandd@cisco.com> | 2009-02-17 14:51:47 -0800 |
| commit | 71d98b4628ee869d62814f6d8607d76cab4b9ec5 (patch) | |
| tree | 10b4d085aaf23020c052d5e340330d757730d138 /drivers/infiniband/hw/ipath | |
| parent | Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband (diff) | |
| download | linux-dev-71d98b4628ee869d62814f6d8607d76cab4b9ec5.tar.xz linux-dev-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 'drivers/infiniband/hw/ipath')
0 files changed, 0 insertions, 0 deletions