[PATCH [mtd-utils]] use xstrdup in a few more places
Mike Frysinger
vapier at gentoo.org
Wed May 8 19:01:55 EDT 2013
These call sites either assume there is no failure (they deref the
pointer right away), or the exit themselves. Use xstrdup() instead.
Signed-off-by: Mike Frysinger <vapier at gentoo.org>
---
jffs2reader.c | 2 +-
mkfs.jffs2.c | 2 +-
mkfs.ubifs/mkfs.ubifs.c | 4 ++--
nanddump.c | 5 +----
4 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/jffs2reader.c b/jffs2reader.c
index 5231ff0..a62da9a 100644
--- a/jffs2reader.c
+++ b/jffs2reader.c
@@ -694,7 +694,7 @@ struct jffs2_raw_dirent *resolvepath0(char *o, size_t size, uint32_t ino,
return NULL;
}
- pp = path = strdup(p);
+ pp = path = xstrdup(p);
if (*path == '/') {
path++;
diff --git a/mkfs.jffs2.c b/mkfs.jffs2.c
index c1b0f0d..f09c0b2 100644
--- a/mkfs.jffs2.c
+++ b/mkfs.jffs2.c
@@ -469,7 +469,7 @@ static int interpret_table_entry(struct filesystem_entry *root, char *line)
} else {
/* If parent is NULL (happens with device table entries),
* try and find our parent now) */
- tmp = strdup(name);
+ tmp = xstrdup(name);
dir = dirname(tmp);
parent = find_filesystem_entry(root, dir, S_IFDIR);
free(tmp);
diff --git a/mkfs.ubifs/mkfs.ubifs.c b/mkfs.ubifs/mkfs.ubifs.c
index 427e37d..f9977bc 100644
--- a/mkfs.ubifs/mkfs.ubifs.c
+++ b/mkfs.ubifs/mkfs.ubifs.c
@@ -534,7 +534,7 @@ static int get_options(int argc, char**argv)
return err_msg("bad maximum LEB count");
break;
case 'o':
- output = strdup(optarg);
+ output = xstrdup(optarg);
break;
case 'D':
tbl_file = optarg;
@@ -627,7 +627,7 @@ static int get_options(int argc, char**argv)
}
if (optind != argc && !output)
- output = strdup(argv[optind]);
+ output = xstrdup(argv[optind]);
if (!output)
return err_msg("not output device or file specified");
diff --git a/nanddump.c b/nanddump.c
index 85ccd5d..4b3e14d 100644
--- a/nanddump.c
+++ b/nanddump.c
@@ -159,10 +159,7 @@ static void process_options(int argc, char * const argv[])
start_addr = simple_strtoll(optarg, &error);
break;
case 'f':
- if (!(dumpfile = strdup(optarg))) {
- perror("stddup");
- exit(EXIT_FAILURE);
- }
+ dumpfile = xstrdup(optarg);
break;
case 'l':
length = simple_strtoll(optarg, &error);
--
1.8.2.1
More information about the linux-mtd
mailing list