nandwrite: complain on malformed -s NUM

Elsewhere: use common error message. -30 bytes net size change

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/coreutils/printf.c b/coreutils/printf.c
index 3497148e3866ab2fa176bb4496bd38399f8fabde..2cc238439177598b0c1f4018672414b9ad6485b2 100644 (file)
--- a/coreutils/printf.c
+++ b/coreutils/printf.c
@@ -66,7 +66,7 @@ static int multiconvert(const char *arg, void *result, converter convert)
        errno = 0;
        convert(arg, result);
        if (errno) {
-               bb_error_msg("%s: invalid number", arg);
+               bb_error_msg("invalid number '%s'", arg);
                return 1;
        }
        return 0;
@@ -230,7 +230,7 @@ static int get_width_prec(const char *str)
 {
        int v = bb_strtoi(str, NULL, 10);
        if (errno) {
-               bb_error_msg("%s: invalid number", str);
+               bb_error_msg("invalid number '%s'", str);
                v = 0;
        }
        return v;

diff --git a/miscutils/nandwrite.c b/miscutils/nandwrite.c
index d0e4397fbde8a1ae84b60b8466c8d703135576ac..8df0fdc81b15ea662e5c06a8b058979858970abe 100644 (file)
--- a/miscutils/nandwrite.c
+++ b/miscutils/nandwrite.c
@@ -72,6 +72,8 @@ int nandwrite_main(int argc UNUSED_PARAM, char **argv)
        xioctl(fd, MEMGETINFO, &meminfo);
 
        mtdoffset = bb_strtou(opt_s, NULL, 0);
+       if (errno)
+               bb_error_msg_and_die("invalid number '%s'", opt_s);
 
        /* Pull it into a CPU register (hopefully) - smaller code that way */
        meminfo_writesize = meminfo.writesize;

diff --git a/procps/kill.c b/procps/kill.c
index 4ed94b0ec51418a594aef1d7e940bf6161d1e96e..b51d44a70a1837b7dd51779f356f3e804a387b7f 100644 (file)
--- a/procps/kill.c
+++ b/procps/kill.c
@@ -153,7 +153,7 @@ int kill_main(int argc, char **argv)
                                        arg = argv[i];
                                omit = bb_strtoi(arg, NULL, 10);
                                if (errno) {
-                                       bb_error_msg("bad pid '%s'", arg);
+                                       bb_error_msg("invalid number '%s'", arg);
                                        ret = 1;
                                        goto resume;
                                }
@@ -211,7 +211,7 @@ int kill_main(int argc, char **argv)
                        arg++;
                pid = bb_strtoi(arg, NULL, 10);
                if (errno) {
-                       bb_error_msg("bad pid '%s'", arg);
+                       bb_error_msg("invalid number '%s'", arg);
                        errors++;
                } else if (kill(pid, signo) != 0) {
                        bb_perror_msg("can't kill pid %d", (int)pid);

diff --git a/procps/renice.c b/procps/renice.c
index 81ee89cfbf98bcfe09c08fce6a55c9a5a3e0a77c..0a73cbeebc1c735213d0adde2253f4df4c0bd3ae 100644 (file)
--- a/procps/renice.c
+++ b/procps/renice.c
@@ -91,7 +91,7 @@ int renice_main(int argc UNUSED_PARAM, char **argv)
                } else {
                        who = bb_strtou(arg, NULL, 10);
                        if (errno) {
-                               bb_error_msg("bad value: %s", arg);
+                               bb_error_msg("invalid number '%s'", arg);
                                goto HAD_ERROR;
                        }
                }

diff --git a/shell/shell_common.c b/shell/shell_common.c
index 9345005c7fa91e69fd60950c4347224f4de4aa1a..e9effd2d0885108c58b287dc9a6f704a37a8c36d 100644 (file)
--- a/shell/shell_common.c
+++ b/shell/shell_common.c
@@ -422,7 +422,7 @@ shell_builtin_ulimit(char **argv)
                                                else
                                                        val = bb_strtoull(val_str, NULL, 10);
                                                if (errno) {
-                                                       bb_error_msg("bad number");
+                                                       bb_error_msg("invalid number '%s'", val_str);
                                                        return EXIT_FAILURE;
                                                }
                                                val <<= l->factor_shift;