chiark / gitweb /
usb_id: remove unused variable
[elogind.git] / extras / usb_id / usb_id.c
index f5ee7959246a8d3a93653497ad0441988915aba7..36477669c50e56618733958e8568100750d578ae 100644 (file)
@@ -1,26 +1,14 @@
 /*
- * usb_id.c
- *
- * Identify an USB (block) device
+ * usb_id - identify an USB device
  *
  * Copyright (c) 2005 SUSE Linux Products GmbH, Germany
+ *
  * Author:
  *     Hannes Reinecke <hare@suse.de>
  *
- *  This library is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU Lesser General Public License as
- *  published by the Free Software Foundation; either version 2.1 of the
- *  License, or (at your option) any later version.
- *
- *  This library is distributed in the hope that it will be useful, but
- *  WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- *  Lesser General Public License for more details.
- *
- *  You should have received a copy of the GNU Lesser General Public
- *  License along with this library; if not, write to the Free Software
- *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
- *  USA
+ *     This program is free software; you can redistribute it and/or modify it
+ *     under the terms of the GNU General Public License as published by the
+ *     Free Software Foundation version 2 of the License.
  */
 
 #include <stdio.h>
 #include <unistd.h>
 #include <string.h>
 #include <ctype.h>
+#include <fcntl.h>
 #include <errno.h>
+#include <getopt.h>
 
-#include <../../libsysfs/sysfs/libsysfs.h>
-#include "../../udev_utils.h"
-#include "../../logging.h"
+#include "../../udev/udev.h"
 
-#define        MAX_NAME_LEN                    72
-#define        MAX_SERIAL_LEN                  256
-#define BLKGETSIZE64 _IOR(0x12,114,size_t)
+int debug;
 
-#ifdef USE_LOG
-void log_message(int priority, const char *format, ...)
+static void log_fn(struct udev *udev, int priority,
+                  const char *file, int line, const char *fn,
+                  const char *format, va_list args)
 {
-       va_list args;
-       static int udev_log = -1;
-
-       if (udev_log == -1) {
-               const char *value;
-
-               value = getenv("UDEV_LOG");
-               if (value)
-                       udev_log = log_priority(value);
-               else
-                       udev_log = LOG_ERR;
+       if (debug) {
+               fprintf(stderr, "%s: ", fn != NULL ? fn : file);
+               vfprintf(stderr, format, args);
+       } else {
+               vsyslog(priority, format, args);
        }
-
-       if (priority > udev_log)
-               return;
-
-       va_start(args, format);
-       vsyslog(priority, format, args);
-       va_end(args);
 }
-#endif
 
-char sysfs_mnt_path[SYSFS_PATH_MAX];
 static char vendor_str[64];
+static char vendor_str_enc[256];
+static const char *vendor_id = "";
 static char model_str[64];
-static char serial_str[MAX_SERIAL_LEN];
-static char revision_str[16];
-static char type_str[16];
+static char model_str_enc[256];
+static const char *product_id = "";
+static char serial_str[UTIL_NAME_SIZE];
+static char packed_if_str[UTIL_NAME_SIZE];
+static char revision_str[64];
+static char type_str[64];
+static char instance_str[64];
+static const char *ifnum;
+static const char *driver;
 
 static int use_usb_info;
 static int use_num_info;
-static int export;
-static int debug;
 
-static void set_str(char *to, const char *from, size_t count)
+static void set_usb_iftype(char *to, int if_class_num, size_t len)
 {
-       size_t i, j, len;
-
-       /* strip trailing whitespace */
-       len = strnlen(from, count);
-       while (isspace(from[len-1]))
-               len--;
-
-       /* strip leading whitespace */
-       i = 0;
-       while (isspace(from[i]) && (i < len))
-               i++;
-
-       j = 0;
-       while (i < len) {
-               /* substitute multiple whitespace */
-               if (isspace(from[i])) {
-                       while (isspace(from[i]))
-                               i++;
-                       to[j++] = '_';
-               }
-               /* Replace '/' with '.' */
-               if (from[i] == '/') {
-                       to[j++] = '.';
-                       i++;
-                       continue;
-               }
-               /* skip non-printable chars */
-               if (!isalnum(from[i]) && !ispunct(from[i])) {
-                       i++;
-                       continue;
-               }
-               to[j++] = from[i++];
-       }
-       to[j] = '\0';
-}
-
-/*
- * set_usb_iftype
- *
- * Set the type based on the USB interface class
- */
-static void set_usb_iftype(char *to, const char *from, size_t len)
-{
-       int type_num;
-       char *eptr;
        char *type = "generic";
 
-       type_num = strtoul(from, &eptr, 0);
-       if (eptr != from) {
-               switch (type_num) {
-               case 1:
-                       type = "audio";
-                       break;
-               case 3:
-                       type = "hid";
-                       break;
-               case 7:
-                       type = "printer";
-                       break;
-               case 8:
-                       type = "disk";
-                       break;
-               case 2: /* CDC-Control */
-               case 5: /* Physical */
-               case 6: /* Image */
-               case 9: /* HUB */
-               case 0x0a: /* CDC-Data */
-               case 0x0b: /* Chip/Smart Card */
-               case 0x0d: /* Content Security */
-               case 0x0e: /* Video */
-               case 0xdc: /* Diagnostic Device */
-               case 0xe0: /* Wireless Controller */
-               case 0xf2: /* Application-specific */
-               case 0xff: /* Vendor-specific */
-               default:
-                       break;
-               }
+       switch (if_class_num) {
+       case 1:
+               type = "audio";
+               break;
+       case 2: /* CDC-Control */
+               break;
+       case 3:
+               type = "hid";
+               break;
+       case 5: /* Physical */
+               break;
+       case 6:
+               type = "media";
+               break;
+       case 7:
+               type = "printer";
+               break;
+       case 8:
+               type = "storage";
+               break;
+       case 9:
+               type = "hub";
+               break;
+       case 0x0a: /* CDC-Data */
+               break;
+       case 0x0b: /* Chip/Smart Card */
+               break;
+       case 0x0d: /* Content Security */
+               break;
+       case 0x0e:
+               type = "video";
+               break;
+       case 0xdc: /* Diagnostic Device */
+               break;
+       case 0xe0: /* Wireless Controller */
+               break;
+       case 0xfe: /* Application-specific */
+               break;
+       case 0xff: /* Vendor-specific */
+               break;
+       default:
+               break;
        }
        strncpy(to, type, len);
        to[len-1] = '\0';
 }
 
-/*
- * set_usb_ifsybtype
- *
- * Set the type base on the interfaceSubClass.
- * Valid for Mass-Storage devices (type 8) only.
- */
-static int set_usb_ifsubtype(char *to, const char *from, size_t len)
+static int set_usb_mass_storage_ifsubtype(char *to, const char *from, size_t len)
 {
        int type_num = 0;
        char *eptr;
@@ -176,7 +115,7 @@ static int set_usb_ifsubtype(char *to, const char *from, size_t len)
        if (eptr != from) {
                switch (type_num) {
                case 2:
-                       type = "cd";
+                       type = "atapi";
                        break;
                case 3:
                        type = "tape";
@@ -186,62 +125,125 @@ static int set_usb_ifsubtype(char *to, const char *from, size_t len)
                        type = "floppy";
                        break;
                case 1: /* RBC devices */
+                       type = "rbc";
+                       break;
                case 6: /* Transparent SPC-2 devices */
-                       type = "disk";
+                       type = "scsi";
                        break;
                default:
                        break;
                }
        }
-       strncpy(to, type, len);
-       to[len-1] = '\0';
-
+       util_strlcpy(to, type, len);
        return type_num;
 }
 
-static void set_scsi_type(char *to, const char *from, int count)
+static void set_scsi_type(char *to, const char *from, size_t len)
 {
        int type_num;
        char *eptr;
+       char *type = "generic";
 
        type_num = strtoul(from, &eptr, 0);
        if (eptr != from) {
                switch (type_num) {
                case 0:
-                       sprintf(to, "disk");
+               case 0xe:
+                       type = "disk";
                        break;
                case 1:
-                       sprintf(to, "tape");
+                       type = "tape";
                        break;
                case 4:
-                       sprintf(to, "optical");
-                       break;
-               case 5:
-                       sprintf(to, "cd");
-                       break;
                case 7:
-                       sprintf(to, "optical");
-                       break;
-               case 0xe:
-                       sprintf(to, "disk");
-                       break;
                case 0xf:
-                       sprintf(to, "optical");
+                       type = "optical";
+                       break;
+               case 5:
+                       type = "cd";
                        break;
                default:
-                       sprintf(to, "generic");
                        break;
                }
-       } else {
-               sprintf(to, "generic");
        }
+       util_strlcpy(to, type, len);
+}
+
+#define USB_DT_DEVICE                  0x01
+#define USB_DT_INTERFACE               0x04
+
+static int dev_if_packed_info(struct udev_device *dev, char *ifs_str, size_t len)
+{
+       char *filename = NULL;
+       int fd;
+       ssize_t size;
+       unsigned char buf[18 + 65535];
+       unsigned int pos, strpos;
+       struct usb_interface_descriptor {
+               u_int8_t        bLength;
+               u_int8_t        bDescriptorType;
+               u_int8_t        bInterfaceNumber;
+               u_int8_t        bAlternateSetting;
+               u_int8_t        bNumEndpoints;
+               u_int8_t        bInterfaceClass;
+               u_int8_t        bInterfaceSubClass;
+               u_int8_t        bInterfaceProtocol;
+               u_int8_t        iInterface;
+       } __attribute__((packed));
+       int err = 0;
+
+       if (asprintf(&filename, "%s/descriptors", udev_device_get_syspath(dev)) < 0) {
+               err = -1;
+               goto out;
+       }
+       fd = open(filename, O_RDONLY);
+       if (fd < 0) {
+               fprintf(stderr, "error opening USB device 'descriptors' file\n");
+               err = -1;
+               goto out;
+       }
+       size = read(fd, buf, sizeof(buf));
+       close(fd);
+       if (size < 18 || size == sizeof(buf)) {
+               err = -1;
+               goto out;
+       }
+
+       pos = 0;
+       strpos = 0;
+       while (pos < sizeof(buf) && strpos+7 < len) {
+               struct usb_interface_descriptor *desc;
+               char if_str[8];
+
+               desc = (struct usb_interface_descriptor *) &buf[pos];
+               if (desc->bLength < 3)
+                       break;
+               pos += desc->bLength;
+
+               if (desc->bDescriptorType != USB_DT_INTERFACE)
+                       continue;
+
+               if (snprintf(if_str, 8, "%02x%02x%02x:",
+                            desc->bInterfaceClass,
+                            desc->bInterfaceSubClass,
+                            desc->bInterfaceProtocol) != 7)
+                       continue;
+
+               if (strstr(ifs_str, if_str) != NULL)
+                       continue;
+
+               memcpy(&ifs_str[strpos], if_str, 8),
+               strpos += 7;
+       }
+out:
+       free(filename);
+       return err;
 }
 
 /*
  * A unique USB identification is generated like this:
  *
- * 1.) Get the USB device type from DeviceClass, InterfaceClass
- *     and InterfaceSubClass
+ * 1.) Get the USB device type from InterfaceClass and InterfaceSubClass
  * 2.) If the device type is 'Mass-Storage/SPC-2' or 'Mass-Storage/RBC'
  *     use the SCSI vendor and model as USB-Vendor and USB-model.
  * 3.) Otherwise use the USB manufacturer and product as
@@ -255,164 +257,187 @@ static void set_scsi_type(char *to, const char *from, int count)
  * 6.) If the device supplies a serial number, this number
  *     is concatenated with the identification with an underscore '_'.
  */
-static int usb_id(const char *target_path)
+static int usb_id(struct udev_device *dev)
 {
-       struct sysfs_class_device *class_dev; /* of target_path */
-       struct sysfs_class_device *class_dev_parent; /* for partitions */
-       struct sysfs_device *scsi_dev; /* the scsi_device */
-       struct sysfs_device *target_dev;
-       struct sysfs_device *host_dev, *interface_dev, *usb_dev;
-       struct sysfs_attribute *scsi_model, *scsi_vendor, *scsi_type, *scsi_rev;
-       struct sysfs_attribute *usb_model = NULL, *usb_vendor = NULL, *usb_rev, *usb_serial;
-       struct sysfs_attribute *if_class, *if_subclass;
+       struct udev *udev = udev_device_get_udev(dev);
+       struct udev_device *dev_interface = NULL;
+       struct udev_device *dev_usb = NULL;
+       const char *if_class, *if_subclass;
        int if_class_num;
        int protocol = 0;
 
-       class_dev = sysfs_open_class_device_path(target_path);
-       if (!class_dev) {
-               info("open class %s failed: %s", target_path, strerror(errno));
-               return 1;
+       dbg(udev, "syspath %s\n", udev_device_get_syspath(dev));
+
+       /* shortcut, if we are called directly for a "usb_device" type */
+       if (udev_device_get_devtype(dev) != NULL && strcmp(udev_device_get_devtype(dev), "usb_device") == 0) {
+               dev_if_packed_info(dev, packed_if_str, sizeof(packed_if_str));
+               dev_usb = dev;
+               goto fallback;
        }
-       class_dev_parent = sysfs_get_classdev_parent(class_dev);
-       if (class_dev_parent) {
-               scsi_dev = sysfs_get_classdev_device(class_dev_parent);
-       } else {
-               scsi_dev = sysfs_get_classdev_device(class_dev);
+
+       /* usb interface directory */
+       dev_interface = udev_device_get_parent_with_subsystem_devtype(dev, "usb", "usb_interface");
+       if (dev_interface == NULL) {
+               info(udev, "unable to access usb_interface device of '%s'\n",
+                    udev_device_get_syspath(dev));
+               return 1;
        }
 
-       /*
-        * The close of scsi_dev will close class_dev or class_dev_parent.
-        */
+       ifnum = udev_device_get_sysattr_value(dev_interface, "bInterfaceNumber");
+       driver = udev_device_get_sysattr_value(dev_interface, "driver");
 
-       /*
-        * We assume we are called after the device is completely ready,
-        * so we don't have to loop here like udev. (And we are usually
-        * called via udev.)
-        */
-       if (!scsi_dev) {
-               /*
-                * errno is not set if we can't find the device link, so
-                * don't print it out here.
-                */
-               info("Cannot find sysfs device associated with %s", target_path);
+       if_class = udev_device_get_sysattr_value(dev_interface, "bInterfaceClass");
+       if (!if_class) {
+               info(udev, "%s: cannot get bInterfaceClass attribute\n",
+                    udev_device_get_sysname(dev));
                return 1;
        }
 
-       /*
-        * Allow only scsi devices.
-        *
-        * Other block devices can support SG IO, but only ide-cd does, so
-        * for now, don't bother with anything else.
-        */
-       if (strcmp(scsi_dev->bus, "scsi") != 0) {
-               info("%s is not a scsi device", target_path);
-               return 1;
+       if_class_num = strtoul(if_class, NULL, 16);
+       if (if_class_num == 8) {
+               /* mass storage */
+               if_subclass = udev_device_get_sysattr_value(dev_interface, "bInterfaceSubClass");
+               if (if_subclass != NULL)
+                       protocol = set_usb_mass_storage_ifsubtype(type_str, if_subclass, sizeof(type_str)-1);
+       } else {
+               set_usb_iftype(type_str, if_class_num, sizeof(type_str)-1);
        }
 
-       /* target directory */
-       target_dev = sysfs_get_device_parent(scsi_dev);
-       /* host directory */
-       host_dev = sysfs_get_device_parent(target_dev);
-       /* usb interface directory */
-       interface_dev = sysfs_get_device_parent(host_dev);
-       /* usb device directory */
-       usb_dev = sysfs_get_device_parent(interface_dev);
+       info(udev, "%s: if_class %d protocol %d\n",
+            udev_device_get_syspath(dev_interface), if_class_num, protocol);
 
-       if (strcmp(interface_dev->bus, "usb") != 0) {
-               info("%s is not an usb device", target_path);
+       /* usb device directory */
+       dev_usb = udev_device_get_parent_with_subsystem_devtype(dev_interface, "usb", "usb_device");
+       if (!dev_usb) {
+               info(udev, "unable to find parent 'usb' device of '%s'\n",
+                    udev_device_get_syspath(dev));
                return 1;
        }
 
-       if_class = sysfs_get_device_attr(interface_dev, "bInterfaceClass");
-       if (!if_class) {
-               info("%s: cannot get bInterfaceClass attribute", interface_dev->name);
-               return 1;
-       }
-       if_class_num = strtoul(if_class->value, NULL, 16);
-       if (if_class_num != 8) {
-               set_usb_iftype(type_str, if_class->value, sizeof(type_str) - 1);
-               protocol = 0;
-       } else {
-               if_subclass = sysfs_get_device_attr(interface_dev, 
-                                                   "bInterfaceSubClass");
-               protocol = set_usb_ifsubtype(type_str, if_subclass->value, 
-                                            sizeof(type_str) -1 );
-       }
+       /* all interfaces of the device in a single string */
+       dev_if_packed_info(dev_usb, packed_if_str, sizeof(packed_if_str));
+
+       /* mass storage : SCSI or ATAPI */
+       if ((protocol == 6 || protocol == 2) && !use_usb_info) {
+               struct udev_device *dev_scsi;
+               const char *scsi_model, *scsi_vendor, *scsi_type, *scsi_rev;
+               int host, bus, target, lun;
+
+               /* get scsi device */
+               dev_scsi = udev_device_get_parent_with_subsystem_devtype(dev, "scsi", "scsi_device");
+               if (dev_scsi == NULL) {
+                       info(udev, "unable to find parent 'scsi' device of '%s'\n",
+                            udev_device_get_syspath(dev));
+                       goto fallback;
+               }
+               if (sscanf(udev_device_get_sysname(dev_scsi), "%d:%d:%d:%d", &host, &bus, &target, &lun) != 4) {
+                       info(udev, "invalid scsi device '%s'\n", udev_device_get_sysname(dev_scsi));
+                       goto fallback;
+               }
 
-       if (!use_usb_info && protocol == 6) {
                /* Generic SPC-2 device */
-               scsi_vendor = sysfs_get_device_attr(scsi_dev, "vendor");
+               scsi_vendor = udev_device_get_sysattr_value(dev_scsi, "vendor");
                if (!scsi_vendor) {
-                       info("%s: cannot get SCSI vendor attribute", scsi_dev->name);
-                       return 1;
+                       info(udev, "%s: cannot get SCSI vendor attribute\n",
+                            udev_device_get_sysname(dev_scsi));
+                       goto fallback;
                }
-               set_str(vendor_str, scsi_vendor->value, sizeof(vendor_str)-1);
+               udev_util_encode_string(scsi_vendor, vendor_str_enc, sizeof(vendor_str_enc));
+               udev_util_replace_whitespace(scsi_vendor, vendor_str, sizeof(vendor_str)-1);
+               udev_util_replace_chars(vendor_str, NULL);
 
-               scsi_model = sysfs_get_device_attr(scsi_dev, "model");
+               scsi_model = udev_device_get_sysattr_value(dev_scsi, "model");
                if (!scsi_model) {
-                       info("%s: cannot get SCSI model attribute", scsi_dev->name);
-                       return 1;
+                       info(udev, "%s: cannot get SCSI model attribute\n",
+                            udev_device_get_sysname(dev_scsi));
+                       goto fallback;
                }
-               set_str(model_str, scsi_model->value, sizeof(model_str)-1);
+               udev_util_encode_string(scsi_model, model_str_enc, sizeof(model_str_enc));
+               udev_util_replace_whitespace(scsi_model, model_str, sizeof(model_str)-1);
+               udev_util_replace_chars(model_str, NULL);
 
-               scsi_type = sysfs_get_device_attr(scsi_dev, "type");
+               scsi_type = udev_device_get_sysattr_value(dev_scsi, "type");
                if (!scsi_type) {
-                       info("%s: cannot get SCSI type attribute", scsi_dev->name);
-                       return 1;
+                       info(udev, "%s: cannot get SCSI type attribute\n",
+                            udev_device_get_sysname(dev_scsi));
+                       goto fallback;
                }
-               set_scsi_type(type_str, scsi_type->value, sizeof(type_str)-1);
+               set_scsi_type(type_str, scsi_type, sizeof(type_str)-1);
 
-               scsi_rev = sysfs_get_device_attr(scsi_dev, "rev");
+               scsi_rev = udev_device_get_sysattr_value(dev_scsi, "rev");
                if (!scsi_rev) {
-                       info("%s: cannot get SCSI revision attribute", scsi_dev->name);
-                       return 1;
+                       info(udev, "%s: cannot get SCSI revision attribute\n",
+                            udev_device_get_sysname(dev_scsi));
+                       goto fallback;
                }
-               set_str(revision_str, scsi_rev->value, sizeof(revision_str)-1);
+               udev_util_replace_whitespace(scsi_rev, revision_str, sizeof(revision_str)-1);
+               udev_util_replace_chars(revision_str, NULL);
 
+               /*
+                * some broken devices have the same identifiers
+                * for all luns, export the target:lun number
+                */
+               sprintf(instance_str, "%d:%d", target, lun);
        }
 
-       /* Fallback to USB vendor & device */
+fallback:
+       vendor_id = udev_device_get_sysattr_value(dev_usb, "idVendor");
+       product_id = udev_device_get_sysattr_value(dev_usb, "idProduct");
+
+       /* fallback to USB vendor & device */
        if (vendor_str[0] == '\0') {
+               const char *usb_vendor = NULL;
+
                if (!use_num_info)
-                       if (!(usb_vendor = sysfs_get_device_attr(usb_dev, "manufacturer")))
-                               dbg("No USB vendor string found, using idVendor");
+                       usb_vendor = udev_device_get_sysattr_value(dev_usb, "manufacturer");
+
+               if (!usb_vendor)
+                       usb_vendor = vendor_id;
 
                if (!usb_vendor) {
-                       if (!(usb_vendor = sysfs_get_device_attr(usb_dev, "idVendor"))) {
-                               dbg("No USB vendor information available\n");
-                               sprintf(vendor_str,"0000");
-                       }
+                       info(udev, "No USB vendor information available\n");
+                       return 1;
                }
-               set_str(vendor_str,usb_vendor->value, sizeof(vendor_str) - 1);
+               udev_util_encode_string(usb_vendor, vendor_str_enc, sizeof(vendor_str_enc));
+               udev_util_replace_whitespace(usb_vendor, vendor_str, sizeof(vendor_str)-1);
+               udev_util_replace_chars(vendor_str, NULL);
        }
-       
+
        if (model_str[0] == '\0') {
+               const char *usb_model = NULL;
+
                if (!use_num_info)
-                       if (!(usb_model = sysfs_get_device_attr(usb_dev, "product")))
-                               dbg("No USB model string found, using idProduct");
-               
+                       usb_model = udev_device_get_sysattr_value(dev_usb, "product");
+
+               if (!usb_model)
+                       usb_model = product_id;
+
                if (!usb_model) {
-                       if (!(usb_model = sysfs_get_device_attr(usb_dev, "idProduct"))) {
-                               dbg("No USB model information available\n");
-                               sprintf(model_str,"0000");
-                       }
+                       dbg(udev, "No USB model information available\n");
+                       return 1;
                }
-               set_str(model_str, usb_model->value, sizeof(model_str) - 1);
+               udev_util_encode_string(usb_model, model_str_enc, sizeof(model_str_enc));
+               udev_util_replace_whitespace(usb_model, model_str, sizeof(model_str)-1);
+               udev_util_replace_chars(model_str, NULL);
        }
 
        if (revision_str[0] == '\0') {
-               usb_rev = sysfs_get_device_attr(usb_dev, "bcdDevice");
+               const char *usb_rev;
+
+               usb_rev = udev_device_get_sysattr_value(dev_usb, "bcdDevice");
                if (usb_rev) {
-                       set_str(revision_str, usb_rev->value, 
-                               sizeof(revision_str) - 1);
+                       udev_util_replace_whitespace(usb_rev, revision_str, sizeof(revision_str)-1);
+                       udev_util_replace_chars(revision_str, NULL);
                }
        }
 
        if (serial_str[0] == '\0') {
-               usb_serial = sysfs_get_device_attr(usb_dev, "serial");
+               const char *usb_serial;
+
+               usb_serial = udev_device_get_sysattr_value(dev_usb, "serial");
                if (usb_serial) {
-                       set_str(serial_str, usb_serial->value,
-                               sizeof(serial_str) - 1);
+                       udev_util_replace_whitespace(usb_serial, serial_str, sizeof(serial_str)-1);
+                       udev_util_replace_chars(serial_str, NULL);
                }
        }
        return 0;
@@ -420,83 +445,123 @@ static int usb_id(const char *target_path)
 
 int main(int argc, char **argv)
 {
-       int retval;
-       char *devpath;
-       char target_path[MAX_NAME_LEN];
-       int option;
-
-       dbg("argc is %d", argc);
-       if (sysfs_get_mnt_path(sysfs_mnt_path, MAX_NAME_LEN)) {
-               info("sysfs_get_mnt_path failed: %s",
-                       strerror(errno));
-               exit(1);
-       }
-
-       while ((option = getopt(argc, argv, "dnux")) != -1 ) {
-               if (optarg)
-                       dbg("option '%c' arg '%s'", option, optarg);
-               else
-                       dbg("option '%c'", option);
+       static const struct option options[] = {
+               { "usb-info", no_argument, NULL, 'u' },
+               { "num-info", no_argument, NULL, 'n' },
+               { "export", no_argument, NULL, 'x' },
+               { "debug", no_argument, NULL, 'd' },
+               { "help", no_argument, NULL, 'h' },
+               {}
+       };
+       struct udev *udev;
+       struct udev_device *dev = NULL;
+       char syspath[UTIL_PATH_SIZE];
+       const char *devpath;
+       static int export;
+       int retval = 0;
+
+       udev = udev_new();
+       if (udev == NULL)
+               goto exit;
+
+       logging_init("usb_id");
+       udev_set_log_fn(udev, log_fn);
+
+       while (1) {
+               int option;
+
+               option = getopt_long(argc, argv, "dnuxh", options, NULL);
+               if (option == -1)
+                       break;
 
                switch (option) {
                case 'd':
                        debug = 1;
+                       if (udev_get_log_priority(udev) < LOG_INFO)
+                               udev_set_log_priority(udev, LOG_INFO);
                        break;
                case 'n':
-                       use_num_info=1;
-                       use_usb_info=1;
+                       use_num_info = 1;
+                       use_usb_info = 1;
                        break;
                case 'u':
-                       use_usb_info=1;
+                       use_usb_info = 1;
                        break;
                case 'x':
-                       export=1;
+                       export = 1;
                        break;
+               case 'h':
+                       printf("Usage: usb_id [--usb-info] [--num-info] [--export] [--help] <devpath>\n"
+                              "  --usb-info  use usb strings instead\n"
+                              "  --num-info  use numerical values\n"
+                              "  --export    print values as environment keys\n"
+                              "  --help      print this help text\n\n");
                default:
-                       info("Unknown or bad option '%c' (0x%x)", option, option);
                        retval = 1;
-                       break;
+                       goto exit;
                }
        }
 
-       devpath = getenv("DEVPATH");
-       if (devpath) {
-               strncpy(target_path, sysfs_mnt_path, MAX_NAME_LEN);
-               strncat(target_path, devpath, MAX_NAME_LEN);
-       } else {
-               if (optind == argc) {
-                       fprintf(stderr, "No device specified\n");
-                       exit(1);
-               }
-               devpath = argv[optind];
-               strncpy(target_path, devpath, MAX_NAME_LEN);
+       devpath = argv[optind];
+       if (devpath == NULL) {
+               fprintf(stderr, "No device specified\n");
+               retval = 1;
+               goto exit;
        }
 
-       retval = usb_id(target_path);
+       util_strlcpy(syspath, udev_get_sys_path(udev), sizeof(syspath));
+       util_strlcat(syspath, devpath, sizeof(syspath));
+       dev = udev_device_new_from_syspath(udev, syspath);
+       if (dev == NULL) {
+               err(udev, "unable to access '%s'\n", devpath);
+               return 1;
+       }
 
+       retval = usb_id(dev);
        if (retval == 0) {
+               char serial[256];
+
+               util_strlcpy(serial, vendor_str, sizeof(serial));
+               util_strlcat(serial, "_", sizeof(serial));
+               util_strlcat(serial, model_str, sizeof(serial));
+               if (serial_str[0] != '\0') {
+                       util_strlcat(serial, "_", sizeof(serial));
+                       util_strlcat(serial, serial_str, sizeof(serial));
+               }
+               if (instance_str[0] != '\0') {
+                       util_strlcat(serial, "-", sizeof(serial));
+                       util_strlcat(serial, instance_str, sizeof(serial));
+               }
+
                if (export) {
                        printf("ID_VENDOR=%s\n", vendor_str);
+                       printf("ID_VENDOR_ENC=%s\n", vendor_str_enc);
+                       printf("ID_VENDOR_ID=%s\n", vendor_id);
                        printf("ID_MODEL=%s\n", model_str);
+                       printf("ID_MODEL_ENC=%s\n", model_str_enc);
+                       printf("ID_MODEL_ID=%s\n", product_id);
                        printf("ID_REVISION=%s\n", revision_str);
-                       if (serial_str[0] == '\0') {
-                               printf("ID_SERIAL=%s_%s\n", 
-                                      vendor_str, model_str);
-                       } else {
-                               printf("ID_SERIAL=%s_%s_%s\n", 
-                                      vendor_str, model_str, serial_str);
-                       }
-                       printf("ID_TYPE=%s\n", type_str);
+                       printf("ID_SERIAL=%s\n", serial);
+                       if (serial_str[0] != '\0')
+                               printf("ID_SERIAL_SHORT=%s\n", serial_str);
+                       if (type_str[0] != '\0')
+                               printf("ID_TYPE=%s\n", type_str);
+                       if (instance_str[0] != '\0')
+                               printf("ID_INSTANCE=%s\n", instance_str);
                        printf("ID_BUS=usb\n");
-               } else {
-                       if (serial_str[0] == '\0') {
-                               printf("%s_%s\n", 
-                                      vendor_str, model_str);
-                       } else {
-                               printf("%s_%s_%s\n", 
-                                      vendor_str, model_str, serial_str);
-                       }
-               }
+                       if (packed_if_str[0] != '\0')
+                               printf("ID_USB_INTERFACES=:%s\n", packed_if_str);
+                       if (ifnum != NULL)
+                               printf("ID_USB_INTERFACE_NUM=%s\n", ifnum);
+                       if (driver != NULL)
+                               printf("ID_USB_DRIVER=%s\n", driver);
+               } else
+                       printf("%s\n", serial);
        }
-       exit(retval);
+
+exit:
+       udev_device_unref(dev);
+       udev_unref(udev);
+       logging_close();
+       return retval;
 }