sctp_diag: Respect ss adding TCPF_CLOSE to idiag_states
authorPhil Sutter <phil@nwl.cc>
Thu, 4 Aug 2016 10:11:57 +0000 (12:11 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 8 Aug 2016 19:51:58 +0000 (12:51 -0700)
Since 'ss' always adds TCPF_CLOSE to idiag_states flags, sctp_diag can't
rely upon TCPF_LISTEN flag solely being present when listening sockets
are requested.

Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/sctp_diag.c

index f728ef04a7b2de394eaefa5347f1bb99944aad87..bb691538adc8e89d6920f8684d8a64779ea1865a 100644 (file)
@@ -356,7 +356,7 @@ static int sctp_ep_dump(struct sctp_endpoint *ep, void *p)
        if (cb->args[4] < cb->args[1])
                goto next;
 
-       if ((r->idiag_states & ~TCPF_LISTEN) && !list_empty(&ep->asocs))
+       if (!(r->idiag_states & TCPF_LISTEN) && !list_empty(&ep->asocs))
                goto next;
 
        if (r->sdiag_family != AF_UNSPEC &&
@@ -471,7 +471,7 @@ skip:
         * 3 : to mark if we have dumped the ep info of the current asoc
         * 4 : to work as a temporary variable to traversal list
         */
-       if (!(idiag_states & ~TCPF_LISTEN))
+       if (!(idiag_states & ~(TCPF_LISTEN | TCPF_CLOSE)))
                goto done;
        sctp_for_each_transport(sctp_tsp_dump, net, cb->args[2], &commp);
 done: