aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorReynaldo H. Verdejo Pinochet <r.verdejo@sisa.samsung.com>2014-04-28 14:02:11 -0400
committerReynaldo H. Verdejo Pinochet <r.verdejo@sisa.samsung.com>2014-04-28 23:54:58 -0400
commit898192e0299ed35e03401fa22df2f7e496b95a52 (patch)
tree4dacf06518579cb11687c5309f34026f1b3dde1a
parente79bc6a88aab8cd85c55461942a421e36a44f5ea (diff)
downloadffmpeg-898192e0299ed35e03401fa22df2f7e496b95a52.tar.gz
ffserver: do not ignore getsockname() return
Should fix FFmpeg Coverity Scan issue #732176 Signed-off-by: Reynaldo H. Verdejo Pinochet <r.verdejo@sisa.samsung.com>
-rw-r--r--ffserver.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/ffserver.c b/ffserver.c
index 20900fb44e..589ba5f331 100644
--- a/ffserver.c
+++ b/ffserver.c
@@ -1759,7 +1759,10 @@ static int http_parse_request(HTTPContext *c)
q += strlen(q);
len = sizeof(my_addr);
- getsockname(c->fd, (struct sockaddr *)&my_addr, &len);
+
+ /* XXX: Should probably fail? */
+ if (getsockname(c->fd, (struct sockaddr *)&my_addr, &len))
+ http_log("getsockname() failed\n");
/* XXX: should use a dynamic buffer */
sdp_data_size = prepare_sdp_description(stream,