diff options
author | Daniel Borkmann <daniel@iogearbox.net> | 2017-01-04 20:34:28 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-01-04 23:55:42 -0500 |
commit | 57ea884b0dcf1e59661955919976ef138ec9cdb0 (patch) | |
tree | 3903d94507481ffceb4514f46233141f4fc82ded /net/packet | |
parent | 5952758101fb55844957a8d4fe88402d9827cfb4 (diff) |
packet: fix panic in __packet_set_timestamp on tpacket_v3 in tx mode
When TX timestamping is in use with TPACKET_V3's TX ring, then we'll
hit the BUG() in __packet_set_timestamp() when ring buffer slot is
returned to user space via tpacket_destruct_skb(). This is due to v3
being assumed as unreachable here, but since 7f953ab2ba46 ("af_packet:
TX_RING support for TPACKET_V3") it's not anymore. Fix it by filling
the timestamp back into the ring slot.
Fixes: 7f953ab2ba46 ("af_packet: TX_RING support for TPACKET_V3")
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/packet')
-rw-r--r-- | net/packet/af_packet.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 7e39087d519b..ddbda255b6ae 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c | |||
@@ -481,6 +481,9 @@ static __u32 __packet_set_timestamp(struct packet_sock *po, void *frame, | |||
481 | h.h2->tp_nsec = ts.tv_nsec; | 481 | h.h2->tp_nsec = ts.tv_nsec; |
482 | break; | 482 | break; |
483 | case TPACKET_V3: | 483 | case TPACKET_V3: |
484 | h.h3->tp_sec = ts.tv_sec; | ||
485 | h.h3->tp_nsec = ts.tv_nsec; | ||
486 | break; | ||
484 | default: | 487 | default: |
485 | WARN(1, "TPACKET version not supported.\n"); | 488 | WARN(1, "TPACKET version not supported.\n"); |
486 | BUG(); | 489 | BUG(); |