summaryrefslogtreecommitdiffstats
path: root/util.c
diff options
context:
space:
mode:
authorNeil Brown <neilb@suse.de>2006-06-20 10:01:43 +1000
committerNeil Brown <neilb@suse.de>2006-06-20 10:01:43 +1000
commite81cdd9f3794d2b4452ae2297c43e16e3faf6f6a (patch)
tree8a2d56d9ab7674857eea75df7249ab83caa30734 /util.c
parent722966c66dd2c95dbe7c194acb66e7c9baf255ec (diff)
downloadmdadm-e81cdd9f3794d2b4452ae2297c43e16e3faf6f6a.tar.gz
mdadm-e81cdd9f3794d2b4452ae2297c43e16e3faf6f6a.tar.xz
mdadm-e81cdd9f3794d2b4452ae2297c43e16e3faf6f6a.zip
Stop map_dev from returning [0:0]
We sometimes need the NULL when major==minor==0. So make sure all callers of map_dev can cope with NULL.
Diffstat (limited to 'util.c')
-rw-r--r--util.c13
1 files changed, 7 insertions, 6 deletions
diff --git a/util.c b/util.c
index d54d445..2cb6dd3 100644
--- a/util.c
+++ b/util.c
@@ -417,12 +417,9 @@ char *map_dev(int major, int minor, int create)
char *std = NULL, *nonstd=NULL;
int did_check = 0;
- if (major == 0 && minor == 0) {
- if (!create)
+ if (major == 0 && minor == 0)
return NULL;
- else
- return "0:0";
- }
+
retry:
if (!devlist_ready) {
char *dev = "/dev";
@@ -636,8 +633,12 @@ int dev_open(char *dev, int flags)
char *e;
int fd = -1;
char devname[32];
- int major = strtoul(dev, &e, 0);
+ int major;
int minor;
+
+ if (!dev) return -1;
+
+ major = strtoul(dev, &e, 0);
if (e > dev && *e == ':' && e[1] &&
(minor = strtoul(e+1, &e, 0)) >= 0 &&
*e == 0) {