From 90c7e32b8009bc7ff85c281f1b02b36828d59cbc Mon Sep 17 00:00:00 2001 From: Vlad Yasevich Date: Tue, 12 Apr 2011 15:20:48 +0000 Subject: [PATCH] sctp: fix oops when updating retransmit path with DEBUG on commit 1ee8fdb17d6b19dd95eed62255e2aa6f27f59e73 sctp: Do no select unconfirmed transports for retransmissions Introduced the initial falt. commit e79770166b701f7ad4229a537ca858d98a24ca5b sctp: Make sure we always return valid retransmit path Solved the problem, but forgot to change the DEBUG statement. Thus it was still possible to dereference a NULL pointer. Signed-off-by: Wei Yongjun Signed-off-by: Vlad Yasevich Signed-off-by: David S. Miller --- net/sctp/associola.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/sctp/associola.c b/net/sctp/associola.c index 0698cad617636..922fdd7eb5733 100644 --- a/net/sctp/associola.c +++ b/net/sctp/associola.c @@ -1323,6 +1323,8 @@ void sctp_assoc_update_retran_path(struct sctp_association *asoc) if (t) asoc->peer.retran_path = t; + else + t = asoc->peer.retran_path; SCTP_DEBUG_PRINTK_IPADDR("sctp_assoc_update_retran_path:association" " %p addr: ", -- 2.39.5