]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "tipc: Fix potential tipc_node refcnt leak in tipc_rcv"
authorTuong Lien <tuong.t.lien@dektech.com.au>
Tue, 2 Jun 2020 04:46:40 +0000 (11:46 +0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 2 Jun 2020 22:13:46 +0000 (15:13 -0700)
commitbeeb38e917253ae0dbae629e87f6f8c9e91ad9d0
tree7fca7334b7c3d61af4558afda17c449768673240
parent6266dee7e113d3cdbc1fbc5f17bbd39e79c825d1
Revert "tipc: Fix potential tipc_node refcnt leak in tipc_rcv"

This reverts commit cdaea38960fd36eec8a06f139c40f9f2f9e2d2a7.

There is no actual tipc_node refcnt leak as stated in the above commit.
The refcnt is hold carefully for the case of an asynchronous decryption
(i.e. -EINPROGRESS/-EBUSY and skb = NULL is returned), so that the node
object cannot be freed in the meantime. The counter will be re-balanced
when the operation's callback arrives with the decrypted buffer if any.
In other cases, e.g. a synchronous crypto the counter will be decreased
immediately when it is done.

Now with that commit, a kernel panic will occur when there is no node
found (i.e. n = NULL) in the 'tipc_rcv()' or a premature release of the
node object.

This commit solves the issues by reverting the said commit, but keeping
one valid case that the 'skb_linearize()' is failed.

Acked-by: Jon Maloy <jmaloy@redhat.com>
Signed-off-by: Tuong Lien <tuong.t.lien@dektech.com.au>
Tested-by: Hoang Le <hoang.h.le@dektech.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/node.c