]> git.baikalelectronics.ru Git - kernel.git/commit
tipc: fix the big/little endian issue in tipc_dest
authorHaiqing Bai <Haiqing.Bai@windriver.com>
Mon, 27 Aug 2018 01:32:26 +0000 (09:32 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 27 Aug 2018 22:23:31 +0000 (15:23 -0700)
commitcbcd9ec601c4a33956d46ca023bbbd2410f55291
tree4d2e12f2daa913eb6c8a7d7908f41dd0c2a42cff
parent153f092a3496df75c95a71f8db64c975d9bc02cf
tipc: fix the big/little endian issue in tipc_dest

In function tipc_dest_push, the 32bit variables 'node' and 'port'
are stored separately in uppper and lower part of 64bit 'value'.
Then this value is assigned to dst->value which is a union like:
union
{
  struct {
    u32 port;
    u32 node;
  };
  u64 value;
}
This works on little-endian machines like x86 but fails on big-endian
machines.

The fix remove the 'value' stack parameter and even the 'value'
member of the union in tipc_dest, assign the 'node' and 'port' member
directly with the input parameter to avoid the endian issue.

Fixes: 6fb4322aaa8c ("tipc: improve destination linked list")
Signed-off-by: Zhenbo Gao <zhenbo.gao@windriver.com>
Acked-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: Haiqing Bai <Haiqing.Bai@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/name_table.c
net/tipc/name_table.h