summaryrefslogtreecommitdiffstats
path: root/include/linux/if_ether.h
diff options
context:
space:
mode:
authorPavel Roskin <proski@gnu.org>2005-11-10 13:03:08 -0800
committerDavid S. Miller <davem@davemloft.net>2005-11-10 13:03:08 -0800
commitac57d04267383829ce817e522e4ebbb39f50ae75 (patch)
tree5a011cf76a5aa8698d33fbb80655561e31e8ac47 /include/linux/if_ether.h
parentfb286bb2990a107009dbf25f6ffebeb7df77f9be (diff)
downloadop-kernel-dev-ac57d04267383829ce817e522e4ebbb39f50ae75.zip
op-kernel-dev-ac57d04267383829ce817e522e4ebbb39f50ae75.tar.gz
[NET]: Annotate h_proto in struct ethhdr
The protocol field in ethernet headers is big-endian and should be annotated as such. This patch allows detection of missing ntohs() calls on the ethernet protocol field when sparse is run with __CHECK_ENDIAN__ defined. This is a revised version that includes <linux/types.h> so that the userspace programs are not confused by __be16. Thanks to David S. Miller. Signed-off-by: Pavel Roskin <proski@gnu.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/if_ether.h')
-rw-r--r--include/linux/if_ether.h4
1 files changed, 3 insertions, 1 deletions
diff --git a/include/linux/if_ether.h b/include/linux/if_ether.h
index d21c305..fe26d43 100644
--- a/include/linux/if_ether.h
+++ b/include/linux/if_ether.h
@@ -21,6 +21,8 @@
#ifndef _LINUX_IF_ETHER_H
#define _LINUX_IF_ETHER_H
+#include <linux/types.h>
+
/*
* IEEE 802.3 Ethernet magic constants. The frame sizes omit the preamble
* and FCS/CRC (frame check sequence).
@@ -100,7 +102,7 @@
struct ethhdr {
unsigned char h_dest[ETH_ALEN]; /* destination eth addr */
unsigned char h_source[ETH_ALEN]; /* source ether addr */
- unsigned short h_proto; /* packet type ID field */
+ __be16 h_proto; /* packet type ID field */
} __attribute__((packed));
#ifdef __KERNEL__
OpenPOWER on IntegriCloud