diff options
author | Peter Maydell <peter.maydell@linaro.org> | 2017-07-14 12:12:25 +0100 |
---|---|---|
committer | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2017-07-15 14:28:25 +0200 |
commit | db0dd60b8c8de91ef3f6d66a2d0c2b62ab94c400 (patch) | |
tree | 9a2625ddd3a13e06c0d4dcdec9420db0e13953c6 /sbuf.c | |
parent | e8f9abd2efacdee175dc811b982234fd9aec28d5 (diff) | |
download | slirp-db0dd60b8c8de91ef3f6d66a2d0c2b62ab94c400.zip slirp-db0dd60b8c8de91ef3f6d66a2d0c2b62ab94c400.tar.gz slirp-db0dd60b8c8de91ef3f6d66a2d0c2b62ab94c400.tar.bz2 |
slirp: Handle error returns from sosendoob()v2.10.0-rc1v2.10.0-rc0
sosendoob() can return a failure code, but all its callers ignore it.
This is OK in sbappend(), as the comment there states -- we will try
again later in sowrite(). Add a (void) cast to tell Coverity so.
In sowrite() we do need to check the return value -- we should handle
a write failure in sosendoob() the same way we handle a write failure
for the normal data.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Diffstat (limited to 'sbuf.c')
-rw-r--r-- | sbuf.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -88,7 +88,7 @@ void sbappend(struct socket *so, struct mbuf *m) if (so->so_urgc) { sbappendsb(&so->so_rcv, m); m_free(m); - sosendoob(so); + (void)sosendoob(so); return; } |