[PATCH] packet.7: fix tpacket_auxdata fields.

[PATCH] packet.7: fix tpacket_auxdata fields.

[kevin sztern]

The definition of the tpacket_auxdata struct in the manpage is not the
same as the definition found in /include/uapi/linux/if_packet.h.

In particular, instead of a tp_padding field, there is a tp_vlan_tpid
field. An example of a project using this field is libpcap[1].

[1]: https://github.com/the-tcpdump-group/libpcap/blob/master/pcap-linux.c#L349
---
 man7/packet.7 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man7/packet.7 b/man7/packet.7
index b624d47d1..ecbd63409 100644
--- a/man7/packet.7
+++ b/man7/packet.7
@@ -257,7 +257,7 @@ struct tpacket_auxdata {
     __u16 tp_mac;
     __u16 tp_net;
     __u16 tp_vlan_tci;
-    __u16 tp_padding;
+    __u16 tp_vlan_tpid;
 };
 .EE
 .in
-- 
2.20.1

Re: [PATCH] packet.7: fix tpacket_auxdata fields.

[Michael Kerrisk (man-pages)]

Hello Kevin,

On 11/27/19 12:22 AM, kevin sztern wrote:
> The definition of the tpacket_auxdata struct in the manpage is not the
> same as the definition found in /include/uapi/linux/if_packet.h.
> 
> In particular, instead of a tp_padding field, there is a tp_vlan_tpid
> field. An example of a project using this field is libpcap[1].
> 
> [1]: https://github.com/the-tcpdump-group/libpcap/blob/master/pcap-linux.c#L349

Thanks. Patch applied.

Cheers,

Michael

> ---
>  man7/packet.7 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man7/packet.7 b/man7/packet.7
> index b624d47d1..ecbd63409 100644
> --- a/man7/packet.7
> +++ b/man7/packet.7
> @@ -257,7 +257,7 @@ struct tpacket_auxdata {
>      __u16 tp_mac;
>      __u16 tp_net;
>      __u16 tp_vlan_tci;
> -    __u16 tp_padding;
> +    __u16 tp_vlan_tpid;
>  };
>  .EE
>  .in
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/