[SCTP] bug: sctp_assoc_lookup_laddr() is broken with ipv6.
authorAl Viro <viro@zeniv.linux.org.uk>
Tue, 21 Nov 2006 01:02:22 +0000 (17:02 -0800)
committerDavid S. Miller <davem@sunset.davemloft.net>
Sun, 3 Dec 2006 05:26:25 +0000 (21:26 -0800)
It expects (and gets) laddr with net-endian sin_port.  And then it calls
sctp_bind_addr_match(), which *does* care about port numbers in case of
ipv6 and expects them to be host-endian.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/associola.c

index a5d7abba89e4d07b5cf541370254273eb1fbcec1..7639044b00749a1c5e0e8d149aa2a4a794142c23 100644 (file)
@@ -1335,10 +1335,12 @@ int sctp_assoc_lookup_laddr(struct sctp_association *asoc,
                            const union sctp_addr *laddr)
 {
        int found;
+       union sctp_addr tmp;
 
+       flip_to_h(&tmp, laddr);
        sctp_read_lock(&asoc->base.addr_lock);
        if ((asoc->base.bind_addr.port == ntohs(laddr->v4.sin_port)) &&
-           sctp_bind_addr_match(&asoc->base.bind_addr, laddr,
+           sctp_bind_addr_match(&asoc->base.bind_addr, &tmp,
                                 sctp_sk(asoc->base.sk))) {
                found = 1;
                goto out;