summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>2007-12-20 22:43:58 +0200
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>2007-12-20 22:45:19 +0200
commitc4d478578c7a1343ad053fca79d80fdc4962ca6f (patch)
tree7b6e10f6ac074ae8891c4238ea61e96695df533a
parentbdf5a0c4ba5a6a022db45a7691ba27c31991f415 (diff)
ubi-tools: fix and cleanup ubirmvol
Plus some more ubimkvol clean-ups Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
-rw-r--r--ubi-utils/Makefile2
-rw-r--r--ubi-utils/src/ubimkvol.c62
-rw-r--r--ubi-utils/src/ubirmvol.c219
3 files changed, 135 insertions, 148 deletions
diff --git a/ubi-utils/Makefile b/ubi-utils/Makefile
index 2baaa07..1a3b2b9 100644
--- a/ubi-utils/Makefile
+++ b/ubi-utils/Makefile
@@ -39,7 +39,7 @@ ubiupdatevol: ubiupdatevol.o error.o libubi.o
ubimkvol: ubimkvol.o common.o libubi.o
$(CC) $(LDFLAGS) -o $@ $^
-ubirmvol: ubirmvol.o error.o libubi.o
+ubirmvol: ubirmvol.o common.o libubi.o
$(CC) $(LDFLAGS) -o $@ $^
pddcustomize: pddcustomize.o error.o libubimirror.o bootenv.o hashmap.o \
diff --git a/ubi-utils/src/ubimkvol.c b/ubi-utils/src/ubimkvol.c
index 2b009d2..454e6fe 100644
--- a/ubi-utils/src/ubimkvol.c
+++ b/ubi-utils/src/ubimkvol.c
@@ -37,9 +37,7 @@
#define PROGRAM_VERSION "1.6"
#define PROGRAM_NAME "ubimkvol"
-/*
- * The variables below are set by command line arguments.
- */
+/* The variables below is set by command line arguments */
struct args {
int devn;
int vol_id;
@@ -79,7 +77,7 @@ static const char *optionsstr =
" eraseblocks\n"
"-m, --maxavsize set volume size to maximum available size\n"
"-t, --type=<static|dynamic> volume type (dynamic, static), default is dynamic\n"
-"-h, --help help message\n"
+"-h, --help print help message\n"
"-V, --version print program version";
static const char *usage =
@@ -91,7 +89,7 @@ static const char *usage =
"Example: " PROGRAM_NAME "/dev/ubi0 -s 20MiB -N config_data - create a 20 Megabytes volume\n"
" named \"config_data\" on UBI device /dev/ubi0.";
-struct option long_options[] = {
+static const struct option long_options[] = {
{ .name = "alignment", .has_arg = 1, .flag = NULL, .val = 'a' },
{ .name = "devn", .has_arg = 1, .flag = NULL, .val = 'd' },
{ .name = "vol_id", .has_arg = 1, .flag = NULL, .val = 'n' },
@@ -102,7 +100,7 @@ struct option long_options[] = {
{ .name = "help", .has_arg = 0, .flag = NULL, .val = 'h' },
{ .name = "version", .has_arg = 0, .flag = NULL, .val = 'V' },
{ .name = "maxavsize", .has_arg = 0, .flag = NULL, .val = 'm' },
- { NULL, 0, NULL, 0}
+ { NULL, 0, NULL, 0},
};
static int parse_opt(int argc, char * const argv[], struct args *args)
@@ -170,7 +168,7 @@ static int parse_opt(int argc, char * const argv[], struct args *args)
return -1;
}
- warnmsg("'-d' and '--devn' options are depricated and will be "
+ warnmsg("'-d' and '--devn' options are deprecated and will be "
"removed. Specify UBI device node name instead!\n"
"Example: " PROGRAM_NAME " /dev/ubi0, instead of "
PROGRAM_NAME " -d 0");
@@ -196,10 +194,6 @@ static int parse_opt(int argc, char * const argv[], struct args *args)
fprintf(stderr, "%s\n", optionsstr);
exit(0);
- case ':':
- errmsg("parameter is missing");
- return -1;
-
case 'V':
fprintf(stderr, "%s\n", PROGRAM_VERSION);
exit(0);
@@ -208,8 +202,12 @@ static int parse_opt(int argc, char * const argv[], struct args *args)
args->maxavs = 1;
break;
+ case ':':
+ errmsg("parameter is missing");
+ return -1;
+
default:
- fprintf(stderr, "Use -h for help");
+ fprintf(stderr, "Use -h for help\n");
exit(-1);
}
}
@@ -219,8 +217,13 @@ static int parse_opt(int argc, char * const argv[], struct args *args)
static int param_sanity_check(struct args *args, libubi_t libubi)
{
- int err, len;
- struct ubi_info ubi;
+ int len;
+
+ if (strlen(args->node) > MAX_NODE_LEN) {
+ errmsg("too long device node name: \"%s\" (%d characters), max. is %d",
+ args->node, strlen(args->node), MAX_NODE_LEN);
+ return -1;
+ }
if (args->bytes == -1 && !args->maxavs && !args->lebs) {
errmsg("volume size was not specified (use -h for help)");
@@ -239,16 +242,21 @@ static int param_sanity_check(struct args *args, libubi_t libubi)
return -1;
}
- err = ubi_get_info(libubi, &ubi);
- if (err) {
- errmsg("cannot get UBI information");
- perror("ubi_get_info");
- return -1;
- }
+ if (args->devn != -1) {
+ int err;
+ struct ubi_info ubi;
- if (args->devn >= (int)ubi.dev_count) {
- errmsg("UBI device %d does not exist", args->devn);
- return -1;
+ err = ubi_get_info(libubi, &ubi);
+ if (err) {
+ errmsg("cannot get UBI information");
+ perror("ubi_get_info");
+ return -1;
+ }
+
+ if (args->devn >= ubi.dev_count) {
+ errmsg("UBI device %d does not exist", args->devn);
+ return -1;
+ }
}
len = strlen(args->name);
@@ -279,13 +287,7 @@ int main(int argc, char * const argv[])
return -1;
}
- if (strlen(argv[1]) > MAX_NODE_LEN) {
- errmsg("too long device node name: \"%s\" (%d characters), max. is %d",
- argv[1], strlen(argv[1]), MAX_NODE_LEN);
- return -1;
- }
-
- strcpy(myargs.node, argv[1]);
+ strncpy(myargs.node, argv[1], MAX_NODE_LEN);
err = parse_opt(argc, (char **)argv, &myargs);
if (err)
diff --git a/ubi-utils/src/ubirmvol.c b/ubi-utils/src/ubirmvol.c
index 6dd16ba..b6fb8ef 100644
--- a/ubi-utils/src/ubirmvol.c
+++ b/ubi-utils/src/ubirmvol.c
@@ -19,13 +19,8 @@
/*
* An utility to remove UBI volumes.
*
- * Author: Artem B. Bityutskiy <dedekind@linutronix.de>
- * Frank Haverkamp <haver@vnet.ibm.com>
- *
- * 1.1 Reworked the userinterface to use argp.
- * 1.2 Removed argp because we want to use uClibc.
- * 1.3 Minor cleanups
- * 1.4 Use a different libubi
+ * Authors: Artem B. Bityutskiy <dedekind@infradead.org>
+ * Frank Haverkamp <haver@vnet.ibm.com>
*/
#include <stdio.h>
@@ -37,119 +32,98 @@
#include <config.h>
#include <libubi.h>
+#include "common.h"
-#define PROGRAM_VERSION "1.4"
+#define PROGRAM_VERSION "1.5"
+#define PROGRAM_NAME "ubirmvol"
-/*
- * The below variables are set by command line options.
- */
+/* The variables below is set by command line arguments */
struct args {
int devn;
int vol_id;
- char node[256];
-
- /* special stuff needed to get additional arguments */
- char *arg1;
- char **options; /* [STRING...] */
+ char node[MAX_NODE_LEN + 1];
};
static struct args myargs = {
.devn = -1,
.vol_id = -1,
-
- .arg1 = NULL,
- .options = NULL,
};
static int param_sanity_check(struct args *args, libubi_t libubi);
-static char doc[] = "\nVersion: " PROGRAM_VERSION "\n"
- "ubirmvol - make UBI Volume.\n";
+static const char *doc = "Version: " PROGRAM_VERSION "\n"
+ PROGRAM_NAME " - a tool to remove UBI volumes.";
static const char *optionsstr =
-" -d, --devn=<devn> UBI device\n"
-" -n, --vol_id=<volume id> UBI volume id, if not specified, the volume ID\n"
-" will be assigned automatically\n"
-" -?, --help Give this help list\n"
-" --usage Give a short usage message\n"
-" -V, --version Print program version\n";
+" -n, --vol_id=<volume id> volume ID to remove\n"
+" -h, --help print help message\n"
+" -V, --version print program version";
static const char *usage =
-"Usage: ubirmvol [-?V] [-d <devn>] [-n <volume id>] [--devn=<devn>]\n"
-" [--vol_id=<volume id>] [--help] [--usage] [--version]\n";
-
-struct option long_options[] = {
- { .name = "devn", .has_arg = 1, .flag = NULL, .val = 'd' },
- { .name = "vol_id", .has_arg = 1, .flag = NULL, .val = 'n' },
- { .name = "help", .has_arg = 0, .flag = NULL, .val = '?' },
- { .name = "usage", .has_arg = 0, .flag = NULL, .val = 0 },
+"Usage: " PROGRAM_NAME " <UBI device node file name> [-n <volume id>] [--vol_id=<volume id>] [-h] [--help]\n\n"
+"Example: " PROGRAM_NAME "/dev/ubi0 -n 1 - remove UBI volume 1 from UBI device corresponding\n"
+" to the node file /dev/ubi0.";
+
+static const struct option long_options[] = {
+ { .name = "devn", .has_arg = 1, .flag = NULL, .val = 'd' },
+ { .name = "vol_id", .has_arg = 1, .flag = NULL, .val = 'n' },
+ { .name = "help", .has_arg = 0, .flag = NULL, .val = 'h' },
{ .name = "version", .has_arg = 0, .flag = NULL, .val = 'V' },
- { NULL, 0, NULL, 0}
+ { NULL, 0, NULL, 0},
};
-/*
- * @brief Parse the arguments passed into the test case.
- *
- * @param argc The number of arguments
- * @param argv The list of arguments
- * @param args Pointer to argument structure
- *
- * @return error
- *
- */
-static int
-parse_opt(int argc, char **argv, struct args *args)
+static int parse_opt(int argc, char * const argv[], struct args *args)
{
char *endp;
while (1) {
int key;
- key = getopt_long(argc, argv, "d:n:?V", long_options, NULL);
+ key = getopt_long(argc, argv, "d:n:hV", long_options, NULL);
if (key == -1)
break;
switch (key) {
- case 'd': /* --devn=<device number> */
- args->devn = strtoul(optarg, &endp, 0);
- if (*endp != '\0' || endp == optarg ||
- args->devn < 0) {
- fprintf(stderr,
- "Bad UBI device number: "
- "\"%s\"\n", optarg);
- goto out;
- }
- sprintf(args->node, "/dev/ubi%d", args->devn);
- break;
- case 'n': /* --volid=<volume id> */
- args->vol_id = strtoul(optarg, &endp, 0);
- if (*endp != '\0' || endp == optarg ||
- (args->vol_id < 0 &&
- args->vol_id != UBI_DYNAMIC_VOLUME)) {
- fprintf(stderr, "Bad volume ID: "
- "\"%s\"\n", optarg);
- goto out;
- }
- break;
- case ':':
- fprintf(stderr, "Parameter is missing\n");
- goto out;
- case '?': /* help */
- fprintf(stderr,
- "Usage: ubirmvol [OPTION...]\n");
- fprintf(stderr, "%s", doc);
- fprintf(stderr, "%s", optionsstr);
- fprintf(stderr, "\nReport bugs to %s\n",
- PACKAGE_BUGREPORT);
- exit(0);
- break;
- case 'V':
- fprintf(stderr, "%s\n", PROGRAM_VERSION);
- exit(0);
- break;
- default:
- fprintf(stderr, "%s", usage);
- exit(-1);
+
+ case 'd':
+ args->devn = strtoul(optarg, &endp, 0);
+ if (*endp != '\0' || endp == optarg || args->devn < 0) {
+ errmsg("bad UBI device number: \"%s\"", optarg);
+ return -1;
+ }
+
+ warnmsg("'-d' and '--devn' options are deprecated and will be "
+ "removed. Specify UBI device node name instead!\n"
+ "Example: " PROGRAM_NAME " /dev/ubi0, instead of "
+ PROGRAM_NAME " -d 0");
+ sprintf(args->node, "/dev/ubi%d", args->devn);
+ break;
+
+ case 'n':
+ args->vol_id = strtoul(optarg, &endp, 0);
+ if (*endp != '\0' || endp == optarg || args->vol_id < 0) {
+ errmsg("bad volume ID: " "\"%s\"", optarg);
+ return -1;
+ }
+ break;
+
+ case 'h':
+ fprintf(stderr, "%s\n\n", doc);
+ fprintf(stderr, "%s\n\n", usage);
+ fprintf(stderr, "%s\n", optionsstr);
+ exit(0);
+
+ case 'V':
+ fprintf(stderr, "%s\n", PROGRAM_VERSION);
+ exit(0);
+
+ case ':':
+ errmsg("parameter is missing");
+ goto out;
+
+ default:
+ fprintf(stderr, "Use -h for help\n");
+ exit(-1);
}
}
@@ -161,62 +135,73 @@ parse_opt(int argc, char **argv, struct args *args)
static int param_sanity_check(struct args *args, libubi_t libubi)
{
int err;
- struct ubi_info ubi;
- if (args->vol_id == -1) {
- fprintf(stderr, "Volume ID was not specified\n");
- goto out;
+ if (strlen(args->node) > MAX_NODE_LEN) {
+ errmsg("too long device node name: \"%s\" (%d characters), max. is %d",
+ args->node, strlen(args->node), MAX_NODE_LEN);
+ return -1;
}
- err = ubi_get_info(libubi, &ubi);
- if (err)
+ if (args->vol_id == -1) {
+ errmsg("volume ID is was not specified");
return -1;
+ }
+
+ if (args->devn != -1) {
+ struct ubi_info ubi;
+
+ err = ubi_get_info(libubi, &ubi);
+ if (err) {
+ errmsg("cannot get UBI information");
+ perror("ubi_get_info");
+ return -1;
+ }
- if (args->devn >= (int)ubi.dev_count) {
- fprintf(stderr, "Device %d does not exist\n", args->devn);
- goto out;
+ if (args->devn >= ubi.dev_count) {
+ errmsg("UBI device %d does not exist", args->devn);
+ return -1;
+ }
}
return 0;
-
-out:
- errno = EINVAL;
- return -1;
}
int main(int argc, char * const argv[])
{
- int err, old_errno;
+ int err;
libubi_t libubi;
- err = parse_opt(argc, (char **)argv, &myargs);
+ strncpy(myargs.node, argv[1], MAX_NODE_LEN);
+
+ err = parse_opt(argc, argv, &myargs);
if (err)
- return err == 1 ? 0 : -1;
+ return -1;
+
+ if (argc < 2) {
+ errmsg("UBI device name was not specified (use -h for help)");
+ return -1;
+ }
- if (myargs.devn == -1) {
- fprintf(stderr, "Device number was not specified\n");
- fprintf(stderr, "Use -h option for help\n");
+ if (argc < 3) {
+ errmsg("too few arguments (use -h for help)");
return -1;
}
libubi = libubi_open();
if (libubi == NULL) {
- perror("Cannot open libubi");
+ errmsg("cannot open libubi");
+ perror("libubi_open");
return -1;
}
err = param_sanity_check(&myargs, libubi);
- if (err) {
- perror("Input parameters check");
- fprintf(stderr, "Use -h option for help\n");
+ if (err)
goto out_libubi;
- }
err = ubi_rmvol(libubi, myargs.node, myargs.vol_id);
- old_errno = errno;
- if (err < 0) {
- perror("Cannot remove volume");
- fprintf(stderr, " err=%d errno=%d\n", err, old_errno);
+ if (err) {
+ errmsg("cannot UBI remove volume");
+ perror("ubi_rmvol");
goto out_libubi;
}