wimax/i2400m: fix bad race condition check in RX path
authorInaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
Thu, 20 May 2010 19:27:58 +0000 (12:27 -0700)
committerInaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
Thu, 20 May 2010 19:27:58 +0000 (12:27 -0700)
commit3a24934f065d23145f1c9c70da9f630c7a37795f
tree0fac89df5f2749e21a45d2c07cf99d16e05ef2fc
parent0fb0a4f00aaf5de9f328273d7a46e3aa27dab496
wimax/i2400m: fix bad race condition check in RX path

The i2400m->rx_roq data structure is protected against race conditions
with a reference count (i2400m->rx_roq_refcount); the pointer can be
read-referenced under the i2400m->rx_lock spinlock.

The code in i2400m_rx_edata() wasn't properly following access
protocol, performing an invalid check on i2400m->rx_roq (which is
cleared to NULL when the refcount drops to zero). As such, it was
missing to detect when the data structure is no longer valid and
oopsing with a NULL pointer dereference.

This commit fixes said check by verifying, under the rx_lock spinlock,
that i2400m->rx_roq is non-NULL and then increasing the reference
count before dropping the spinlock.

Signed-off-by: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
drivers/net/wimax/i2400m/rx.c