Commit 7c17fcc7 authored Oct 31, 2016 by Xin Long Committed by David S. Miller Oct 31, 2016

sctp: return back transport in __sctp_rcv_init_lookup



Prior to this patch, it used a local variable to save the transport that is
looked up by __sctp_lookup_association(), and didn't return it back. But in
sctp_rcv, it is used to initialize chunk->transport. So when hitting this,
even if it found the transport, it was still initializing chunk->transport
with null instead.

This patch is to return the transport back through transport pointer
that is from __sctp_rcv_lookup_harder().

Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent cd26da4f

net/sctp/input.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -1021,7 +1021,6 @@ static struct sctp_association __sctp_rcv_init_lookup(struct net net,
		struct sctphdr *sh = sctp_hdr(skb);
		union sctp_params params;
		sctp_init_chunk_t *init;
		struct sctp_transport *transport;
		struct sctp_af *af;

		/*
		@@ -1052,7 +1051,7 @@ static struct sctp_association __sctp_rcv_init_lookup(struct net net,

		af->from_addr_param(paddr, params.addr, sh->source, 0);

		asoc = __sctp_lookup_association(net, laddr, paddr, &transport);
		asoc = __sctp_lookup_association(net, laddr, paddr, transportp);
		if (asoc)
		return asoc;
		}

Admin message