datapath: Make 'length' local variable unsigned, for consistency.

This is a style issue, not a bug, if you chase down what the function
and the caller are doing.

Found by Chris Eagle via Fortify.

diff --git a/datapath/datapath.c b/datapath/datapath.c
index e5ed9df1b405cf3199396d731d2b14a08c707f73..1043742e23d2094b4c447609f8a82a9870b6140d 100644 (file)
--- a/datapath/datapath.c
+++ b/datapath/datapath.c
@@ -602,7 +602,7 @@ static void dp_frame_hook(struct sk_buff *skb)
 
 static inline unsigned packet_length(const struct sk_buff *skb)
 {
-       int length = skb->len - ETH_HLEN;
+       unsigned length = skb->len - ETH_HLEN;
        if (skb->protocol == htons(ETH_P_8021Q))
                length -= VLAN_HLEN;
        return length;