summaryrefslogtreecommitdiff
path: root/serprog.c
diff options
context:
space:
mode:
authorstefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1>2014-04-26 16:12:31 +0000
committerstefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1>2014-04-26 16:12:31 +0000
commit16c34902d0899a51fabd730cc6a9417c12e355a3 (patch)
treed53f8c4d58abaf1c61e36d8300e46219be2b5a32 /serprog.c
parentc30299065455be105b1008796369da020e0a55e7 (diff)
downloadflashrom-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.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/serprog.c b/serprog.c
index 86c67e0..35c4f32 100644
--- a/serprog.c
+++ b/serprog.c
@@ -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