diff options
| author |   hayeswang <hayeswang@realtek.com> | 2017-03-14 14:15:20 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-03-21 14:37:10 -0700 |
| commit | 98d068ab52b4b11d403995ed14154660797e7136 (patch) | |
| tree | d09a0090cb530375e5eeae54af6123d7f332e0ab | |
| parent | tcp: tcp_get_info() should read tcp_time_stamp later (diff) | |
| download | linux-dev-98d068ab52b4b11d403995ed14154660797e7136.tar.xz linux-dev-98d068ab52b4b11d403995ed14154660797e7136.zip | |
r8152: fix the list rx_done may be used without initialization
The list rx_done would be initialized when the linking on occurs.
Therefore, if a napi is scheduled without any linking on before,
the following kernel panic would happen.
BUG: unable to handle kernel NULL pointer dereference at 000000000000008
IP: [<ffffffffc085efde>] r8152_poll+0xe1e/0x1210 [r8152]
PGD 0
Oops: 0002 [#1] SMP
Signed-off-by: Hayes Wang <hayeswang@realtek.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/usb/r8152.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 986243c932cc..bb3eedd07fbe 100644 --- a/drivers/net/usb/r8152.c +++ b/drivers/net/usb/r8152.c @@ -1362,6 +1362,7 @@ static int alloc_all_mem(struct r8152 *tp) spin_lock_init(&tp->rx_lock); spin_lock_init(&tp->tx_lock); INIT_LIST_HEAD(&tp->tx_free); + INIT_LIST_HEAD(&tp->rx_done); skb_queue_head_init(&tp->tx_queue); skb_queue_head_init(&tp->rx_queue); |