diff options
author | Tobias Klauser <tklauser@distanz.ch> | 2013-06-14 14:34:28 +0200 |
---|---|---|
committer | Tobias Klauser <tklauser@distanz.ch> | 2013-06-14 14:34:28 +0200 |
commit | e07a1ee28df19afc6cd7360010b6ba862d29505f (patch) | |
tree | 53fa4c53599ca483fb5840b177497ebb531a5842 | |
parent | 8cb741a7dce0bcc9140a1f717028a8b4c80559f5 (diff) |
sock: Check return value of setsockopt() in set_tcp_nodelay()
Check the return value of setsockopt() and panic in case it is not 0, as
in the other setsockopt() wrapper functions.
This issue was discovered by the Coverity scanner.
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
-rw-r--r-- | sock.c | 7 |
1 files changed, 5 insertions, 2 deletions
@@ -65,8 +65,11 @@ void set_socket_keepalive(int fd) void set_tcp_nodelay(int fd) { - int one = 1; - setsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &one, sizeof(one)); + int ret, one = 1; + + ret = setsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &one, sizeof(one)); + if (unlikely(ret)) + panic("Cannot set TCP nodelay!\n"); } int set_ipv6_only(int fd) |