[FFmpeg-cvslog] ffserver: do not ignore send() return

Reynaldo H. Verdejo Pinochet git at videolan.org
Tue Apr 29 07:12:13 CEST 2014


ffmpeg | branch: master | Reynaldo H. Verdejo Pinochet <r.verdejo at sisa.samsung.com> | Mon Apr 28 13:45:34 2014 -0400| [e79bc6a88aab8cd85c55461942a421e36a44f5ea] | committer: Reynaldo H. Verdejo Pinochet

ffserver: do not ignore send() return

Should fix Coverity Scan issue #732178

Signed-off-by: Reynaldo H. Verdejo Pinochet <r.verdejo at sisa.samsung.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e79bc6a88aab8cd85c55461942a421e36a44f5ea
---

 ffserver.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ffserver.c b/ffserver.c
index 1d32892..20900fb 100644
--- a/ffserver.c
+++ b/ffserver.c
@@ -811,7 +811,8 @@ static void http_send_too_busy_reply(int fd)
                        "</body></html>\r\n",
                        nb_connections, nb_max_connections);
     av_assert0(len < sizeof(buffer));
-    send(fd, buffer, len, 0);
+    if (send(fd, buffer, len, 0) < len)
+        av_log(NULL, AV_LOG_WARNING, "Could not send too-busy reply, send() failed\n");
 }
 
 



More information about the ffmpeg-cvslog mailing list