diff options
author | Vlad Yasevich <vladislav.yasevich@hp.com> | 2011-04-12 11:20:48 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-04-12 22:33:50 -0400 |
commit | 25f7bf7d0dfb460505cbe42676340e33100aca2e (patch) | |
tree | 76467af7ca483f836f9acc2fc73f000db6223a4e /net | |
parent | 31d8b9e099e59f880aa65095951559896d4e20fa (diff) |
sctp: fix oops when updating retransmit path with DEBUG on
commit fbdf501c9374966a56829ecca3a7f25d2b49a305
sctp: Do no select unconfirmed transports for retransmissions
Introduced the initial falt.
commit d598b166ced20d9b9281ea3527c0e18405ddb803
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 <yjwei@cn.fujitsu.com>
Signed-off-by: Vlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/sctp/associola.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/sctp/associola.c b/net/sctp/associola.c index 0698cad61763..922fdd7eb573 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) | |||
1323 | 1323 | ||
1324 | if (t) | 1324 | if (t) |
1325 | asoc->peer.retran_path = t; | 1325 | asoc->peer.retran_path = t; |
1326 | else | ||
1327 | t = asoc->peer.retran_path; | ||
1326 | 1328 | ||
1327 | SCTP_DEBUG_PRINTK_IPADDR("sctp_assoc_update_retran_path:association" | 1329 | SCTP_DEBUG_PRINTK_IPADDR("sctp_assoc_update_retran_path:association" |
1328 | " %p addr: ", | 1330 | " %p addr: ", |