[vlc-commits] Zero is a legal file descriptor

Rémi Denis-Courmont git at videolan.org
Mon Jul 25 20:51:31 CEST 2016


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Mon Jul 25 19:31:32 2016 +0300| [7010722262970c678f762e60bf89df70e6feee7a] | committer: Rémi Denis-Courmont

Zero is a legal file descriptor

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=7010722262970c678f762e60bf89df70e6feee7a
---

 modules/access/mms/mmsh.c | 4 ++--
 modules/access/satip.c    | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/modules/access/mms/mmsh.c b/modules/access/mms/mmsh.c
index 9571974..32a09ab 100644
--- a/modules/access/mms/mmsh.c
+++ b/modules/access/mms/mmsh.c
@@ -729,7 +729,7 @@ static int Describe( access_t  *p_access, char **ppsz_location )
     return VLC_SUCCESS;
 
 error:
-    if( p_sys->fd > 0 )
+    if( p_sys->fd >= 0 )
     {
         net_Close( p_sys->fd  );
         p_sys->fd = -1;
@@ -890,7 +890,7 @@ static void Stop( access_t *p_access )
     access_sys_t *p_sys = p_access->p_sys;
 
     msg_Dbg( p_access, "closing stream" );
-    if( p_sys->fd > 0 )
+    if( p_sys->fd >= 0 )
     {
         net_Close( p_sys->fd );
         p_sys->fd = -1;
diff --git a/modules/access/satip.c b/modules/access/satip.c
index 715260b..cb3a82f 100644
--- a/modules/access/satip.c
+++ b/modules/access/satip.c
@@ -774,11 +774,11 @@ static void satip_close(access_t *access) {
     }
 #endif
 
-    if (sys->udp_sock > 0)
+    if (sys->udp_sock >= 0)
         net_Close(sys->udp_sock);
-    if (sys->rtcp_sock > 0)
+    if (sys->rtcp_sock >= 0)
         net_Close(sys->rtcp_sock);
-    if (sys->tcp_sock > 0)
+    if (sys->tcp_sock >= 0)
         net_Close(sys->tcp_sock);
 
     free(sys->content_base);



More information about the vlc-commits mailing list