Signed-off-by: Alexey Dobriyan <[EMAIL PROTECTED]>
---

 Also, there is the following snippet in net/ipx/af_ipx.c, line 1174

        f.ipx_special = sipx->sipx_special;

 The former is unsigned char and is compared with 1 and 2. The latter is
 __be16 if I'm not mistaken. Is there a bug hiding?

--- a/net/ipx/ipx_route.c
+++ b/net/ipx/ipx_route.c
@@ -238,7 +237,7 @@ int ipxrtr_route_packet(struct sock *sk,
        }       
 
        /* Apply checksum. Not allowed on 802.3 links. */
-       if (sk->sk_no_check || intrfc->if_dlink_type == IPX_FRAME_8023)
+       if (sk->sk_no_check || intrfc->if_dlink_type == htons(IPX_FRAME_8023))
                ipx->ipx_checksum = 0xFFFF;
        else
                ipx->ipx_checksum = ipx_cksum(ipx, len + sizeof(struct ipxhdr));

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to