upnp: remove m_server_list_lock, p_server_list is already protected by s_lock
[vlc.git] / modules / services_discovery / udev.c
index 736bdb6..1df75fd 100644 (file)
@@ -27,6 +27,8 @@
 #include <errno.h>
 #include <search.h>
 #include <poll.h>
+#include <sys/stat.h>
+#include <fcntl.h>
 
 #include <libudev.h>
 
@@ -36,6 +38,7 @@
 #ifdef HAVE_ALSA
 # include <vlc_modules.h>
 #endif
+#include <vlc_fs.h>
 #include <vlc_url.h>
 
 static int OpenV4L (vlc_object_t *);
@@ -135,11 +138,12 @@ struct services_discovery_sys_t
 static int cmpdev (const void *a, const void *b)
 {
     const dev_t *da = a, *db = b;
-    dev_t delta = *da - *db;
 
-    if (sizeof (delta) > sizeof (int))
-        return delta ? ((delta > 0) ? 1 : -1) : 0;
-    return (signed)delta;
+    if (*da > *db)
+        return +1;
+    if (*da < *db)
+        return -1;
+    return 0;
 }
 
 static void DestroyDevice (void *data)
@@ -165,9 +169,8 @@ static int AddDevice (services_discovery_t *sd, struct udev_device *dev)
     if (mrl == NULL)
         return 0; /* don't know if it was an error... */
     char *name = p_sys->subsys->get_name (dev);
-    input_item_t *item = input_item_NewWithType (mrl, name ? name : mrl,
-                                                 0, NULL, 0, -1,
-                                                 p_sys->subsys->item_type);
+    input_item_t *item = input_item_NewExt (mrl, name ? name : mrl, -1,
+                                            p_sys->subsys->item_type, ITEM_LOCAL);
     msg_Dbg (sd, "adding %s (%s)", mrl, name);
     free (name);
     free (mrl);
@@ -533,8 +536,9 @@ static char *disc_get_mrl (struct udev_device *dev)
     val = udev_device_get_property_value (dev, "ID_CDROM_MEDIA_STATE");
     if (val == NULL)
     {   /* Force probing of the disc in the drive if any. */
-        int fd = open (node, O_RDONLY);
-        close (fd);
+        int fd = vlc_open (node, O_RDONLY);
+        if (fd != -1)
+            vlc_close (fd);
         return NULL;
     }
     if (!strcmp (val, "blank"))