tree 3b72ba5f585fcab8271dc4be046b3bd447852847
parent 1d89b1ae6c203bc68a3f424be144abcdf62773c9
author Eli Cohen <eli@mellanox.co.il> 1141102063 -0800
committer Roland Dreier <rolandd@cisco.com> 1142878098 -0800

IPoIB: Close race in setting mcast->ah

ipoib_mcast_send() tests mcast->ah twice.  If this value is changed
between these two points, we leak an skb.  However,
ipoib_mcast_join_finish() sets mcast->ah with no locking, so it could
race against ipoib_mcast_send().

As a solution, take priv->lock around assignment to mcast->ah thus
making sure ipoib_mcast_send() (which also takes priv->lock) is not in
flight.

Signed-off-by: Eli Cohen <eli@mellanox.co.il>
Signed-off-by: Michael S. Tsirkin <mst@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>