Bluetooth: bnep: use constant for ethertype
authorEldad Zack <eldad@fogrefinery.com>
Mon, 7 May 2012 22:09:35 +0000 (00:09 +0200)
committerGustavo Padovan <gustavo@padovan.org>
Wed, 9 May 2012 04:40:54 +0000 (01:40 -0300)
The dot1q ethertype number (0x8100) is embedded in the code, although
it is already defined in included headers.

Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Signed-off-by: Gustavo Padovan <gustavo@padovan.org>
net/bluetooth/bnep/core.c

index a779ec703323ce7293522e3cdb7af1d41b3c58a2..4fab4362b5aafd362e612bdbbd839caa9cb408c8 100644 (file)
@@ -340,7 +340,7 @@ static inline int bnep_rx_frame(struct bnep_session *s, struct sk_buff *skb)
        }
 
        /* Strip 802.1p header */
-       if (ntohs(s->eh.h_proto) == 0x8100) {
+       if (ntohs(s->eh.h_proto) == ETH_P_8021Q) {
                if (!skb_pull(skb, 4))
                        goto badframe;
                s->eh.h_proto = get_unaligned((__be16 *) (skb->data - 2));