dvdnav: trivially avoid duplicate file path lookup
authorRémi Denis-Courmont <rdenis@simphalempin.com>
Sun, 11 Jan 2009 14:40:13 +0000 (16:40 +0200)
committerRémi Denis-Courmont <rdenis@simphalempin.com>
Sun, 11 Jan 2009 14:42:27 +0000 (16:42 +0200)
modules/access/dvdnav.c

index 953eb23..b1d7863 100644 (file)
@@ -1435,16 +1435,15 @@ static int ProbeDVD( demux_t *p_demux, char *psz_name )
         return VLC_SUCCESS;
     }
 
-    if( stat( psz_name, &stat_info ) || !S_ISREG( stat_info.st_mode ) )
+    if( (i_fd = open( psz_name, O_RDONLY )) == -1 )
     {
-        /* Let dvdnav_open() do the probing */
-        return VLC_SUCCESS;
+        return VLC_SUCCESS; /* Let dvdnav_open() do the probing */
     }
 
-    if( (i_fd = open( psz_name, O_RDONLY )) == -1 )
+    if( fstat( i_fd, &stat_info ) || !S_ISREG( stat_info.st_mode ) )
     {
-        /* Let dvdnav_open() do the probing */
-        return VLC_SUCCESS;
+        close( i_fd );
+        return VLC_SUCCESS; /* Let dvdnav_open() do the probing */
     }
 
     /* Try to find the anchor (2 bytes at LBA 256) */