X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=udev%2Fudevadm-info.c;h=2f41b7adf77cd7bb9165fbbe17c639cc28467695;hb=9921484458e0233e1aae868bfa8cc1481aa460a8;hp=4a9ebcd014b1d7770272ec53a7983d23d1da5d67;hpb=93b0f384589fce818edb9678571a30e71930d14c;p=elogind.git

diff --git a/udev/udevadm-info.c b/udev/udevadm-info.c
index 4a9ebcd01..2f41b7adf 100644
--- a/udev/udevadm-info.c
+++ b/udev/udevadm-info.c
@@ -15,8 +15,6 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "config.h"
-
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
@@ -128,138 +126,43 @@ static int print_device_chain(struct udev_device *device)
 	return 0;
 }
 
-static int print_record_devlinks_cb(struct udev_device *device, const char *value, void *data)
-{
-	size_t len;
-
-	len = strlen(udev_get_dev_path(udev_device_get_udev(device)));
-	printf("S: %s\n", &value[len+1]);
-	return 0;
-}
-
-static int print_record_properties_cb(struct udev_device *device, const char *key, const char *value, void *data)
-{
-	printf("E: %s=%s\n", key, value);
-	return 0;
-}
-
 static void print_record(struct udev_device *device)
 {
 	size_t len;
 	int i;
+	struct udev_list *list;
 
 	printf("P: %s\n", udev_device_get_devpath(device));
+
 	len = strlen(udev_get_dev_path(udev_device_get_udev(device)));
-	printf("N: %s\n", &udev_device_get_devname(device)[len+1]);
+	printf("N: %s\n", &udev_device_get_devnode(device)[len+1]);
+
 	i = device_get_devlink_priority(device);
 	if (i != 0)
 		printf("L: %i\n", i);
+
 	i = device_get_num_fake_partitions(device);
 	if (i != 0)
 		printf("A:%u\n", i);
+
 	i = device_get_ignore_remove(device);
 	if (i != 0)
 		printf("R:%u\n", i);
-	udev_device_get_devlinks(device, print_record_devlinks_cb, NULL);
-	udev_device_get_properties(device, print_record_properties_cb, NULL);
-	printf("\n");
-}
-
-static int export_all_cb(struct udev *udev,
-			 const char *devpath, const char *subsystem, const char *name,
-			 void *data)
-{
-	struct udev_device *device;
 
-	device = udev_device_new_from_devpath(udev, devpath);
-	if (device == NULL)
-		return 0;
-	if (udev_device_get_devname(device) != NULL)
-		print_record(device);
-	udev_device_unref(device);
-	return 0;
-}
-
-static struct udev_device *lookup_device_by_name(struct udev *udev, const char *name)
-{
-#if 0
-	/* FIXME */
-	LIST_HEAD(name_list);
-	int count;
-	struct name_entry *device;
-	int rc  = -1;
-
-	count = udev_db_get_devices_by_name(udev, name, &name_list);
-	if (count <= 0)
-		goto out;
-
-	info(udev, "found %i devices for '%s'\n", count, name);
-
-	/* select the device that seems to match */
-	list_for_each_entry(device, &name_list, node) {
-		struct udevice *udevice_loop;
-		char filename[UTIL_PATH_SIZE];
-		struct stat statbuf;
-
-		udevice_loop = udev_device_init(udev);
-		if (udevice_loop == NULL)
-			break;
-		if (udev_db_get_device(udevice_loop, device->name) != 0)
-			goto next;
-		info(udev, "found db entry '%s'\n", device->name);
-
-		/* make sure, we don't get a link of a different device */
-		util_strlcpy(filename, udev_get_dev_path(udev), sizeof(filename));
-		util_strlcat(filename, "/", sizeof(filename));
-		util_strlcat(filename, name, sizeof(filename));
-		if (stat(filename, &statbuf) != 0)
-			goto next;
-		if (major(udevice_loop->devt) > 0 && udevice_loop->devt != statbuf.st_rdev) {
-			info(udev, "skip '%s', dev_t doesn't match\n", udevice_loop->name);
-			goto next;
-		}
-		rc = 0;
-		*udevice = udevice_loop;
-		break;
-next:
-		udev_device_cleanup(udevice_loop);
+	list = udev_device_get_devlinks_list(device);
+	while (list != NULL) {
+		len = strlen(udev_get_dev_path(udev_device_get_udev(device)));
+		printf("S: %s\n", &udev_list_get_name(list)[len+1]);
+		list = udev_list_get_next(list);
 	}
-out:
-	name_list_cleanup(udev, &name_list);
-	return rc;
-#endif
-	return NULL;
-}
-
-static int add_devlink_cb(struct udev_device *device, const char *value, void *data)
-{
-	char **links = data;
 
-	if (*links == NULL) {
-		*links = strdup(value);
-	} else {
-		char *str;
-
-		asprintf(&str, "%s %s", *links, value);
-		free(*links);
-		*links = str;
+	list = udev_device_get_properties_list(device);
+	while (list != NULL) {
+		printf("E: %s=%s\n", udev_list_get_name(list), udev_list_get_value(list));
+		list = udev_list_get_next(list);
 	}
-	return 0;
-}
-
-static int add_devlink_noroot_cb(struct udev_device *device, const char *value, void *data)
-{
-	size_t len;
-
-	len = strlen(udev_get_dev_path(udev_device_get_udev(device)));
-	value = &value[len+1];
-	return add_devlink_cb(device, value, data);
-}
 
-static int print_property_cb(struct udev_device *device, const char *key, const char *value, void *data)
-{
-	printf("%s=%s\n", key, value);
-	return 0;
+	printf("\n");
 }
 
 static int stat_device(const char *name, int export, const char *prefix)
@@ -281,6 +184,30 @@ static int stat_device(const char *name, int export, const char *prefix)
 	return 0;
 }
 
+static int export_devices(struct udev *udev)
+{
+	struct udev_enumerate *enumerate;
+	struct udev_list *list;
+
+	enumerate = udev_enumerate_new_from_subsystems(udev, NULL);
+	if (enumerate == NULL)
+		return -1;
+	list = udev_enumerate_get_list(enumerate);
+	while (list != NULL) {
+		struct udev_device *device;
+
+		device = udev_device_new_from_syspath(udev, udev_list_get_name(list));
+		if (device != NULL) {
+			if (udev_device_get_devnode(device) != NULL)
+				print_record(device);
+			udev_device_unref(device);
+		}
+		list = udev_list_get_next(list);
+	}
+	udev_enumerate_unref(enumerate);
+	return 0;
+}
+
 int udevadm_info(struct udev *udev, int argc, char *argv[])
 {
 	struct udev_device *device = NULL;
@@ -289,7 +216,7 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 	const char *export_prefix = NULL;
 	char path[UTIL_PATH_SIZE];
 	char name[UTIL_PATH_SIZE];
-	char *links;
+	struct udev_list *list;
 	int rc = 0;
 
 	static const struct option options[] = {
@@ -326,6 +253,7 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 
 	while (1) {
 		int option;
+		struct stat statbuf;
 
 		option = getopt_long(argc, argv, "aed:n:p:q:rxPVh", options, NULL);
 		if (option == -1)
@@ -340,12 +268,37 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 				goto exit;
 			}
 			/* remove /dev if given */
-			if (strncmp(optarg, udev_get_dev_path(udev), strlen(udev_get_dev_path(udev))) == 0)
-				util_strlcpy(name, &optarg[strlen(udev_get_dev_path(udev))+1], sizeof(name));
-			else
+			if (strncmp(optarg, udev_get_dev_path(udev), strlen(udev_get_dev_path(udev))) != 0) {
+				util_strlcpy(name, udev_get_dev_path(udev), sizeof(name));
+				util_strlcat(name, "/", sizeof(name));
+				util_strlcat(name, optarg, sizeof(name));
+			} else {
 				util_strlcpy(name, optarg, sizeof(name));
+			}
 			util_remove_trailing_chars(name, '/');
-			device = lookup_device_by_name(udev, name);
+			if (stat(name, &statbuf) < 0) {
+				fprintf(stderr, "device node not found\n");
+				rc = 2;
+				goto exit;
+			} else {
+				char type;
+
+				if (S_ISBLK(statbuf.st_mode)) {
+					type = 'b';
+				} else if (S_ISCHR(statbuf.st_mode)) {
+					type = 'c';
+				} else {
+					fprintf(stderr, "device node has wrong file type\n");
+					rc = 2;
+					goto exit;
+				}
+				device = udev_device_new_from_devnum(udev, type, statbuf.st_rdev);
+				if (device == NULL) {
+					fprintf(stderr, "device node not found\n");
+					rc = 2;
+					goto exit;
+				}
+			}
 			break;
 		case 'p':
 			if (device != NULL) {
@@ -353,13 +306,20 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 				rc = 2;
 				goto exit;
 			}
-			/* remove /sys if given */
-			if (strncmp(optarg, udev_get_sys_path(udev), strlen(udev_get_sys_path(udev))) == 0)
-				util_strlcpy(path, &optarg[strlen(udev_get_sys_path(udev))], sizeof(path));
-			else
+			/* add /sys if needed */
+			if (strncmp(optarg, udev_get_sys_path(udev), strlen(udev_get_sys_path(udev))) != 0) {
+				util_strlcpy(path, udev_get_sys_path(udev), sizeof(path));
+				util_strlcat(path, optarg, sizeof(path));
+			} else {
 				util_strlcpy(path, optarg, sizeof(path));
+			}
 			util_remove_trailing_chars(path, '/');
-			device = udev_device_new_from_devpath(udev, path);
+			device = udev_device_new_from_syspath(udev, path);
+			if (device == NULL) {
+				fprintf(stderr, "device path not found\n");
+				rc = 2;
+				goto exit;
+			}
 			break;
 		case 'q':
 			action = ACTION_QUERY;
@@ -399,7 +359,7 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 			action = ACTION_ATTRIBUTE_WALK;
 			break;
 		case 'e':
-			udev_enumerate_devices(udev, NULL, export_all_cb, NULL);
+			export_devices(udev);
 			goto exit;
 		case 'x':
 			export = 1;
@@ -415,18 +375,18 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 			goto exit;
 		case 'h':
 			printf("Usage: udevadm info OPTIONS\n"
-			       "  --query=<type>             query database for the specified value:\n"
+			       "  --query=<type>             query device information:\n"
 			       "      name                     name of device node\n"
 			       "      symlink                  pointing to node\n"
-			       "      path                     sysfs device path\n"
+			       "      path                     sys device path\n"
 			       "      env                      the device related imported environment\n"
 			       "      all                      all values\n"
-			       "  --path=<devpath>           sysfs device path used for query or chain\n"
-			       "  --name=<name>              node or symlink name used for query\n"
-			       "  --root                     prepend to query result or print udev_root\n"
-			       "  --attribute-walk           print all key matches while walking along chain\n"
+			       "  --path=<syspath>           sys device path used for query or attribute walk\n"
+			       "  --name=<name>              node or symlink name used for query or attribute walk\n"
+			       "  --root                     prepend dev directory to path names\n"
+			       "  --attribute-walk           print all key matches while walking along the chain\n"
 			       "                             of parent devices\n"
-			       "  --device-id-of-file=<file> print major/minor of underlying device\n"
+			       "  --device-id-of-file=<file> print major:minor of device containing this file\n"
 			       "  --export-db                export the content of the udev database\n"
 			       "  --help                     print this text\n"
 			       "\n");
@@ -439,7 +399,7 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 	switch (action) {
 	case ACTION_QUERY:
 		if (device == NULL) {
-			fprintf(stderr, "query needs --path= or node --name= specified\n");
+			fprintf(stderr, "query needs a valid device specified by --path= or --name=\n");
 			rc = 4;
 			goto exit;
 		}
@@ -447,28 +407,40 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 		switch(query) {
 		case QUERY_NAME:
 			if (root) {
-				printf("%s\n", udev_device_get_devname(device));
+				printf("%s\n", udev_device_get_devnode(device));
 			} else {
 				size_t len;
 
 				len = strlen(udev_get_dev_path(udev));
-				printf("%s\n", &udev_device_get_devname(device)[len+1]);
+				printf("%s\n", &udev_device_get_devnode(device)[len+1]);
 			}
 			break;
 		case QUERY_SYMLINK:
-			links = NULL;
-			if (root)
-				udev_device_get_devlinks(device, add_devlink_cb, &links);
-			else
-				udev_device_get_devlinks(device, add_devlink_noroot_cb, &links);
-			printf("%s\n", links);
-			free(links);
+			list = udev_device_get_devlinks_list(device);
+			while (list != NULL) {
+				if (root) {
+					printf("%s", udev_list_get_name(list));
+				} else {
+					size_t len;
+
+					len = strlen(udev_get_dev_path(udev_device_get_udev(device)));
+					printf("%s", &udev_list_get_name(list)[len+1]);
+				}
+				list = udev_list_get_next(list);
+				if (list != NULL)
+					printf(" ");
+			}
+			printf("\n");
 			break;
 		case QUERY_PATH:
 			printf("%s\n", udev_device_get_devpath(device));
 			goto exit;
 		case QUERY_ENV:
-			udev_device_get_properties(device, print_property_cb, NULL);
+			list = udev_device_get_properties_list(device);
+			while (list != NULL) {
+				printf("%s=%s\n", udev_list_get_name(list), udev_list_get_value(list));
+				list = udev_list_get_next(list);
+			}
 			break;
 		case QUERY_ALL:
 			print_record(device);
@@ -480,7 +452,7 @@ int udevadm_info(struct udev *udev, int argc, char *argv[])
 		break;
 	case ACTION_ATTRIBUTE_WALK:
 		if (device == NULL) {
-			fprintf(stderr, "attribute walk needs --path or node --name specified\n");
+			fprintf(stderr, "query needs a valid device specified by --path= or --name=\n");
 			rc = 5;
 			goto exit;
 		}