tipc: Fix bug in topology server byte swapping routine
authorAllan Stephens <allan.stephens@windriver.com>
Mon, 19 May 2008 20:29:06 +0000 (13:29 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 19 May 2008 20:29:06 +0000 (13:29 -0700)
This patch fixes TIPC's topology server so that it does byte swapping
correctly when endianness conversion is required.  (Note: This bug only
impacted an application if it issues a subscription request to a
topology server on another node, rather than the server on it's own
node; since the topology server is normally not accessible by off-node
applications, most TIPC applications were not impacted by the bug.)

Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/subscr.c

index a62e5d30638c1651aec0183a54d99087497e9e44..dde23f1e7542da377954c4d4a027364c6b6626ab 100644 (file)
@@ -86,9 +86,7 @@ static struct top_srv topsrv = { 0 };
 
 static u32 htohl(u32 in, int swap)
 {
-       char *c = (char *)&in;
-
-       return swap ? ((c[3] << 3) + (c[2] << 2) + (c[1] << 1) + c[0]) : in;
+       return swap ? (u32)___constant_swab32(in) : in;
 }
 
 /**