diff options
| author | Kay Sievers <kay.sievers@vrfy.org> | 2009-03-18 18:42:16 +0100 |
|---|---|---|
| committer | Kay Sievers <kay.sievers@vrfy.org> | 2009-03-18 18:42:16 +0100 |
| commit | f454f6708a1ca439dbe9abd6b9fb660423a71d14 (patch) | |
| tree | 83a343121fa293f7bfa0f272ce6e7c2b28ed74fe /udev/udevadm-test.c | |
| parent | b25d8cfa93c93fb2b98fdd747777294c787c6619 (diff) | |
udevadm: test - handling trailing '/' in devpath
On Wed, Mar 18, 2009 at 16:00, Matthias Schwarzott <zzam@gentoo.org> wrote:
found out how the error occurs:
It is a difference between
A. udevadm test /sys/class/mem/null/
and
B. udevadm test /sys/class/mem/null
Case A was the case that showed the error behaviour. It seems udevadm is
confused by the trailing slash. This behaviour seems to be there since ages.
Diffstat (limited to 'udev/udevadm-test.c')
| -rw-r--r-- | udev/udevadm-test.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/udev/udevadm-test.c b/udev/udevadm-test.c index 664d72f6d3..c53c273295 100644 --- a/udev/udevadm-test.c +++ b/udev/udevadm-test.c @@ -94,12 +94,14 @@ int udevadm_test(struct udev *udev, int argc, char *argv[]) if (strncmp(syspath, udev_get_sys_path(udev), strlen(udev_get_sys_path(udev))) != 0) { util_strlcpy(filename, udev_get_sys_path(udev), sizeof(filename)); util_strlcat(filename, syspath, sizeof(filename)); - syspath = filename; + } else { + util_strlcpy(filename, syspath, sizeof(filename)); } + util_remove_trailing_chars(filename, '/'); - dev = udev_device_new_from_syspath(udev, syspath); + dev = udev_device_new_from_syspath(udev, filename); if (dev == NULL) { - fprintf(stderr, "unable to open device '%s'\n", syspath); + fprintf(stderr, "unable to open device '%s'\n", filename); rc = 2; goto exit; } |