X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=udev-remove.c;h=0f14a3d685f7745e3afa8c05ea2443c8e640138a;hp=9862a6f201baf757cfb91b345dcde8dfef2383f8;hb=f7b4eca455c7dbf850d984892756f22dbd9ddc3d;hpb=ca9998602336f67eb2590819eecf416977885e2a diff --git a/udev-remove.c b/udev-remove.c index 9862a6f20..0f14a3d68 100644 --- a/udev-remove.c +++ b/udev-remove.c @@ -36,27 +36,30 @@ /* - * Here would go a call to the naming deamon, to get the name we want to have - * for this device. But for now, let's just default to whatever the kernel is - * calling the device as that will keep the "old-style" naming policy + * Look up the sysfs path in the database to see if we have named this device + * something different from the kernel name. If we have, us it. If not, use + * the default kernel name for lack of anything else to know to do. */ -static char *get_name(char *dev, int major, int minor) +static char *get_name(char *path, int major, int minor) { static char name[100]; + struct udevice *dev; char *temp; - temp = udevdb_get_udevice_by_sysfs(dev); - dbg("udevdb_get_udevice_by_sysfs returned %s", temp); - if (temp != NULL) - return temp; - - temp = strrchr(dev, '/'); + dev = udevdb_get_dev(path); + if (dev != NULL) { + strcpy(name, dev->name); + goto exit; + } + + dbg("'%s' not found in database, falling back on default name", path); + temp = strrchr(path, '/'); if (temp == NULL) return NULL; strncpy(name, &temp[1], sizeof(name)); - dbg("name is %s", name); - +exit: + dbg("name is '%s'", name); return &name[0]; } @@ -66,12 +69,45 @@ static char *get_name(char *dev, int major, int minor) static int delete_node(char *name) { char filename[255]; + int retval; - strncpy(filename, UDEV_ROOT, sizeof(filename)); + strncpy(filename, udev_root, sizeof(filename)); strncat(filename, name, sizeof(filename)); - dbg("unlinking %s", filename); - return unlink(filename); + dbg("unlinking '%s'", filename); + retval = unlink(filename); + if (retval) { + dbg("unlink(%s) failed with error '%s'", + filename, strerror(errno)); + return retval; + } + + /* remove subdirectories */ + if (strchr(name, '/')) { + char *pos; + + pos = strrchr(filename, '/'); + while (1) { + *pos = 0x00; + pos = strrchr(filename, '/'); + + /* don't remove the last one */ + if ((pos == filename) || (pos == NULL)) + break; + + /* remove if empty */ + retval = rmdir(filename); + if (retval) { + if (errno == ENOTEMPTY) + return 0; + dbg("rmdir(%s) failed with error '%s'", + filename, strerror(errno)); + break; + } + dbg("removed '%s'", filename); + } + } + return retval; } int udev_remove_device(char *device, char *subsystem) @@ -86,7 +122,7 @@ int udev_remove_device(char *device, char *subsystem) goto exit; } - udevdb_delete_udevice(name); + udevdb_delete_dev(device); return delete_node(name);