]> Pileus Git - ~andy/linux/commit
tipc: don't reroute message fragments
authorErik Hugne <erik.hugne@ericsson.com>
Wed, 6 Nov 2013 08:28:05 +0000 (09:28 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 7 Nov 2013 23:30:11 +0000 (18:30 -0500)
commit528f6f4bf372511ebf6004eed822de8f749b5930
tree3db1786d51ec6f55d468f1a0e9028b61119ee4ba
parentb0db7b0c21a014d01be1018db68e78ebf7d4f0d7
tipc: don't reroute message fragments

When a message fragment is received in a broadcast or unicast link,
the reception code will append the fragment payload to a big reassembly
buffer through a call to the function tipc_recv_fragm(). However, after
the return of that call, the logics goes on and passes the fragment
buffer to the function tipc_net_route_msg(), which will simply drop it.
This behavior is a remnant from the now obsolete multi-cluster
functionality, and has no relevance in the current code base.

Although currently harmless, this unnecessary call would be fatal
after applying the next patch in this series, which introduces
a completely new reassembly algorithm. So we change the code to
eliminate the redundant call.

Signed-off-by: Erik Hugne <erik.hugne@ericsson.com>
Reviewed-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/bcast.c
net/tipc/link.c