diff options
author | stefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2014-04-26 16:12:31 +0000 |
---|---|---|
committer | stefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2014-04-26 16:12:31 +0000 |
commit | 16c34902d0899a51fabd730cc6a9417c12e355a3 (patch) | |
tree | d53f8c4d58abaf1c61e36d8300e46219be2b5a32 /serprog.c | |
parent | c30299065455be105b1008796369da020e0a55e7 (diff) | |
download | flashrom-16c34902d0899a51fabd730cc6a9417c12e355a3.tar.gz |
CID1129996: Unchecked return value from setsockopt() in serprog.
Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Acked-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
git-svn-id: https://code.coreboot.org/svn/flashrom/trunk@1777 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'serprog.c')
-rw-r--r-- | serprog.c | 6 |
1 files changed, 5 insertions, 1 deletions
@@ -115,7 +115,11 @@ static int sp_opensocket(char *ip, unsigned int port) } /* We are latency limited, and sometimes do write-write-read * * (write-n) - so enable TCP_NODELAY. */ - setsockopt(sock, IPPROTO_TCP, TCP_NODELAY, &flag, sizeof(int)); + if (setsockopt(sock, IPPROTO_TCP, TCP_NODELAY, &flag, sizeof(int))) { + close(sock); + msg_perr("Error: serprog cannot set socket options: %s\n", strerror(errno)); + return -1; + } return sock; } #endif |