diff options
author | Rob Landley <rob@landley.net> | 2006-01-23 02:14:16 +0300 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2006-01-23 02:14:16 +0300 |
commit | e927743bb022996e9f5c75d3ef20696fc5049dbf (patch) | |
tree | f71daa487c79705a1f47f766cd2bdafba6e159c0 /util-linux/mdev.c | |
parent | 86f5c9906beac527f3d7966f24811b571a589e08 (diff) |
There was a missing ! on strcmp (despite the fact it worked in my testing,
I have no idea _how_), and a simpler fix for the extra \n problem. I tested
this and it worked for me, for what that's worth...
Diffstat (limited to 'util-linux/mdev.c')
-rw-r--r-- | util-linux/mdev.c | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/util-linux/mdev.c b/util-linux/mdev.c index 135843581..a18e44835 100644 --- a/util-linux/mdev.c +++ b/util-linux/mdev.c @@ -43,23 +43,13 @@ static void make_device(char *path) fd = open(temp, O_RDONLY); len = read(fd, temp, PATH_MAX-1); if (len<1) goto end; - temp[--len] = 0; // remove trailing \n close(fd); /* Determine device name, type, major and minor */ device_name = strrchr(path, '/') + 1; type = strncmp(path+5, "block/" ,6) ? S_IFCHR : S_IFBLK; - major = minor = 0; - for (s = temp; *s; s++) { - if (*s == ':') { - major = minor; - minor = 0; - } else { - minor *= 10; - minor += (*s) - '0'; - } - } + if(sscanf(temp, "%d:%d", &major, &minor) != 2) goto end; /* If we have a config file, look up permissions for this device */ @@ -202,7 +192,7 @@ static void find_dev(char *path) /* If there's a dev entry, mknod it */ - if (strcmp(entry->d_name, "dev")) make_device(path); + if (!strcmp(entry->d_name, "dev")) make_device(path); } closedir(dir); |