sctp: do not mark sk dumped when inet_sctp_diag_fill returns err
authorXin Long <lucien.xin@gmail.com>
Fri, 15 Sep 2017 03:02:48 +0000 (11:02 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 15 Sep 2017 21:51:15 +0000 (14:51 -0700)
commit8c7c19a55e41ae69d1cd18ab56e6e9b66a679a7c
treee41b66148295d87976db3bd0f1514bc97fd86da8
parentd25adbeb0cdb860fb39e09cdd025e9cfc954c5ab
sctp: do not mark sk dumped when inet_sctp_diag_fill returns err

sctp_diag would not actually dump out sk/asoc if inet_sctp_diag_fill
returns err, in which case it shouldn't mark sk dumped by setting
cb->args[3] as 1 in sctp_sock_dump().

Otherwise, it could cause some asocs to have no parent's sk dumped
in 'ss --sctp'.

So this patch is to not set cb->args[3] when inet_sctp_diag_fill()
returns err in sctp_sock_dump().

Fixes: 8f840e47f190 ("sctp: add the sctp_diag.c file")
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/sctp_diag.c