summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Fenkart <andreas.fenkart@digitalstrom.com>2015-12-09 13:13:25 +0100
committerTom Rini <trini@konsulko.com>2016-02-08 10:10:32 -0500
commit1ce686978c03d4fd025c7e93b7a18d4a87f547ec (patch)
tree85be48a2c124ff643bf84fd714f480dd49a82a19
parent371ee13760b9e70d1a931afc643de3f4057b4a28 (diff)
downloadu-boot-imx-1ce686978c03d4fd025c7e93b7a18d4a87f547ec.zip
u-boot-imx-1ce686978c03d4fd025c7e93b7a18d4a87f547ec.tar.gz
u-boot-imx-1ce686978c03d4fd025c7e93b7a18d4a87f547ec.tar.bz2
tools: env: shift optind arguments and fix argument indices
Signed-off-by: Andreas Fenkart <andreas.fenkart@digitalstrom.com>
-rw-r--r--tools/env/fw_env.c54
-rw-r--r--tools/env/fw_env_main.c4
2 files changed, 15 insertions, 43 deletions
diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
index bd195b4..ee17a69 100644
--- a/tools/env/fw_env.c
+++ b/tools/env/fw_env.c
@@ -239,22 +239,10 @@ int fw_printenv (int argc, char *argv[])
char *env, *nxt;
int i, rc = 0;
-#ifdef CONFIG_FILE
- if (argc >= 2 && strcmp(argv[1], "-c") == 0) {
- argv += 2;
- argc -= 2;
- }
-#endif
-
- if (argc >= 2 && strcmp(argv[1], "-a") == 0) {
- argv += 2;
- argc -= 2;
- }
-
if (fw_env_open())
return -1;
- if (argc == 1) { /* Print all env variables */
+ if (argc == 0) { /* Print all env variables */
for (env = environment.data; *env; env = nxt + 1) {
for (nxt = env; *nxt; ++nxt) {
if (nxt >= &environment.data[ENV_SIZE]) {
@@ -269,17 +257,13 @@ int fw_printenv (int argc, char *argv[])
return 0;
}
- if (strcmp (argv[1], "-n") == 0) {
- ++argv;
- --argc;
- if (argc != 2) {
- fprintf (stderr, "## Error: "
- "`-n' option requires exactly one argument\n");
- return -1;
- }
+ if (printenv_args.name_suppress && argc != 1) {
+ fprintf(stderr,
+ "## Error: `-n' option requires exactly one argument\n");
+ return -1;
}
- for (i = 1; i < argc; ++i) { /* print single env variables */
+ for (i = 0; i < argc; ++i) { /* print single env variables */
char *name = argv[i];
char *val = NULL;
@@ -476,24 +460,8 @@ int fw_setenv(int argc, char *argv[])
char *value = NULL;
int valc;
-#ifdef CONFIG_FILE
- if (argc >= 2 && strcmp(argv[1], "-c") == 0) {
- argv += 2;
- argc -= 2;
- }
-#endif
-
- if (argc < 2) {
- errno = EINVAL;
- return -1;
- }
-
- if (strcmp(argv[1], "-a") == 0) {
- argv += 2;
- argc -= 2;
- }
-
- if (argc < 2) {
+ if (argc < 1) {
+ fprintf(stderr, "## Error: variable name missing\n");
errno = EINVAL;
return -1;
}
@@ -503,9 +471,9 @@ int fw_setenv(int argc, char *argv[])
return -1;
}
- name = argv[1];
- valv = argv + 2;
- valc = argc - 2;
+ name = argv[0];
+ valv = argv + 1;
+ valc = argc - 1;
if (env_flags_validate_env_set_params(name, valv, valc) < 0)
return 1;
diff --git a/tools/env/fw_env_main.c b/tools/env/fw_env_main.c
index b68f1bf..daf4688 100644
--- a/tools/env/fw_env_main.c
+++ b/tools/env/fw_env_main.c
@@ -182,6 +182,10 @@ int main(int argc, char *argv[])
exit(EXIT_FAILURE);
}
+ /* shift parsed flags, jump to non-option arguments */
+ argc -= optind;
+ argv += optind;
+
lockfd = open(lockname, O_WRONLY | O_CREAT | O_TRUNC, 0666);
if (-1 == lockfd) {
fprintf(stderr, "Error opening lock file %s\n", lockname);