diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-02-12 11:57:32 +0100 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-02-12 12:04:53 +0100 |
commit | cb86f2cd33a845872198c4bb207bc13e739b2d7e (patch) | |
tree | e5ca71c69d67fae396edfd5e8fd7971dc5e2ce3f /src/peer.h | |
parent | d39c586ee03387e9d03f4dff1715fcd309738145 (diff) |
socket: free skb if there isn't an endpoint
Because the packet transmission interface in Linux, in general, has the
characteristic of "consuming" skbs, our own socket_send family too must
always consume skbs. This means that when a caller passes an skb to a
socket_send function, he looses ownership and expects for the skb to be
freed by the socket_send function, not by himself. Therefore, it's
important that we actually carry out this contract under all
circumstances, even when there's no endpoint available.
This condition could be hit by setting up a peer with a persistent
keepalive but no endpoint, or by removing an endpoint once packets were
queued up to be sent already.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'src/peer.h')
0 files changed, 0 insertions, 0 deletions