diff options
author | Cong Wang <xiyou.wangcong@gmail.com> | 2018-03-26 15:08:33 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-05-30 07:52:20 +0200 |
commit | 5b5f4fd97d8fdc5d41ad632ed5154989b38b613f (patch) | |
tree | 5d689e5c2f0f566ce60afcada965d2a821426757 /include/linux/lz4.h | |
parent | 25801736ca4818040b184c5547e6f5a14eaf4847 (diff) | |
download | linux-stable-5b5f4fd97d8fdc5d41ad632ed5154989b38b613f.tar.gz linux-stable-5b5f4fd97d8fdc5d41ad632ed5154989b38b613f.tar.bz2 linux-stable-5b5f4fd97d8fdc5d41ad632ed5154989b38b613f.zip |
llc: properly handle dev_queue_xmit() return value
[ Upstream commit b85ab56c3f81c5a24b5a5213374f549df06430da ]
llc_conn_send_pdu() pushes the skb into write queue and
calls llc_conn_send_pdus() to flush them out. However, the
status of dev_queue_xmit() is not returned to caller,
in this case, llc_conn_state_process().
llc_conn_state_process() needs hold the skb no matter
success or failure, because it still uses it after that,
therefore we should hold skb before dev_queue_xmit() when
that skb is the one being processed by llc_conn_state_process().
For other callers, they can just pass NULL and ignore
the return value as they are.
Reported-by: Noam Rathaus <noamr@beyondsecurity.com>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/lz4.h')
0 files changed, 0 insertions, 0 deletions