diff options
| author | Xin Long | 2016-09-13 20:04:23 +0200 |
|---|---|---|
| committer | David S. Miller | 2016-09-19 04:02:33 +0200 |
| commit | 41001faf95faaff7c4f4f93c6bb544ee227ad0cc (patch) | |
| tree | ac2f30f99428c982ffecfd5f40e94407ae2f704d /net/ax25/ax25_ds_timer.c | |
| parent | sctp: make sctp_outq_flush/tail/uncork return void (diff) | |
| download | kernel-qcow2-linux-41001faf95faaff7c4f4f93c6bb544ee227ad0cc.tar.gz kernel-qcow2-linux-41001faf95faaff7c4f4f93c6bb544ee227ad0cc.tar.xz kernel-qcow2-linux-41001faf95faaff7c4f4f93c6bb544ee227ad0cc.zip | |
sctp: not return ENOMEM err back in sctp_packet_transmit
As David and Marcelo's suggestion, ENOMEM err shouldn't return back to
user in transmit path. Instead, sctp's retransmit would take care of
the chunks that fail to send because of ENOMEM.
This patch is only to do some release job when alloc_skb fails, not to
return ENOMEM back any more.
Besides, it also cleans up sctp_packet_transmit's err path, and fixes
some issues in err path:
- It didn't free the head skb in nomem: path.
- No need to check nskb in no_route: path.
- It should goto err: path if alloc_skb fails for head.
- Not all the NOMEMs should free nskb.
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ax25/ax25_ds_timer.c')
0 files changed, 0 insertions, 0 deletions