RDMA/rxe: Fix coding error in rxe_rcv_mcast_pkt
authorBob Pearson <rpearsonhpe@gmail.com>
Thu, 28 Jan 2021 17:47:53 +0000 (11:47 -0600)
committerJason Gunthorpe <jgg@nvidia.com>
Fri, 5 Feb 2021 17:44:22 +0000 (13:44 -0400)
commit8fc1b7027fc162738d5a85c82410e501a371a404
tree7127b5006a1b51ad67c4aca2f9144fc8ac7b5399
parente328197423e09094aff48619ebef6671ff64d3b2
RDMA/rxe: Fix coding error in rxe_rcv_mcast_pkt

rxe_rcv_mcast_pkt() in rxe_recv.c can leak SKBs in error path code. The
loop over the QPs attached to a multicast group creates new cloned SKBs
for all but the last QP in the list and passes the SKB and its clones to
rxe_rcv_pkt() for further processing. Any QPs that do not pass some checks
are skipped.  If the last QP in the list fails the tests the SKB is
leaked.  This patch checks if the SKB for the last QP was used and if not
frees it. Also removes a redundant loop invariant assignment.

Fixes: 8700e3e7c485 ("Soft RoCE driver")
Fixes: 71abf20b28ff ("RDMA/rxe: Handle skb_clone() failure in rxe_recv.c")
Link: https://lore.kernel.org/r/20210128174752.16128-1-rpearson@hpe.com
Signed-off-by: Bob Pearson <rpearson@hpe.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/sw/rxe/rxe_recv.c