summaryrefslogtreecommitdiffstats
path: root/net/unix
diff options
context:
space:
mode:
authorEric Dumazet <edumazet@google.com>2015-10-23 10:59:16 -0700
committerDavid S. Miller <davem@davemloft.net>2015-10-25 06:37:45 -0700
commit1586a5877db9eee313379738d6581bc7c6ffb5e3 (patch)
tree67dfe980f1ce3a826883360f51babeabdb73c820 /net/unix
parent742e038330a485350334ee5eb75dce4a9dff87cd (diff)
downloadop-kernel-dev-1586a5877db9eee313379738d6581bc7c6ffb5e3.zip
op-kernel-dev-1586a5877db9eee313379738d6581bc7c6ffb5e3.tar.gz
af_unix: do not report POLLOUT on listeners
poll(POLLOUT) on a listener should not report fd is ready for a write(). This would break some applications using poll() and pfd.events = -1, as they would not block in poll() Signed-off-by: Eric Dumazet <edumazet@google.com> Reported-by: Alan Burlison <Alan.Burlison@oracle.com> Tested-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/unix')
-rw-r--r--net/unix/af_unix.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/net/unix/af_unix.c b/net/unix/af_unix.c
index 94f6582..aaa0b58 100644
--- a/net/unix/af_unix.c
+++ b/net/unix/af_unix.c
@@ -326,9 +326,10 @@ found:
return s;
}
-static inline int unix_writable(struct sock *sk)
+static int unix_writable(const struct sock *sk)
{
- return (atomic_read(&sk->sk_wmem_alloc) << 2) <= sk->sk_sndbuf;
+ return sk->sk_state != TCP_LISTEN &&
+ (atomic_read(&sk->sk_wmem_alloc) << 2) <= sk->sk_sndbuf;
}
static void unix_write_space(struct sock *sk)
OpenPOWER on IntegriCloud