From patchwork Tue Nov 24 13:21:11 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Fenkart X-Patchwork-Id: 548064 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 3A3701401E7 for ; Wed, 25 Nov 2015 00:31:26 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b=LNZiINVu; dkim-atps=neutral Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 458BC4BD6C; Tue, 24 Nov 2015 14:31:03 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 5NqozC2iX9KE; Tue, 24 Nov 2015 14:31:03 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 989A84BD6E; Tue, 24 Nov 2015 14:30:41 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 895C64B9C4 for ; Tue, 24 Nov 2015 14:21:37 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id T2cfbYxUYAxB for ; Tue, 24 Nov 2015 14:21:37 +0100 (CET) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by theia.denx.de (Postfix) with ESMTPS id 3D0954B976 for ; Tue, 24 Nov 2015 14:21:34 +0100 (CET) Received: by wmww144 with SMTP id w144so26127320wmw.0 for ; Tue, 24 Nov 2015 05:21:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=JM5lODN5tI4vcR8VyZcIVc88E4lXxBYZi78DP/F0wJE=; b=LNZiINVuHAUlLo7FeokzlGvHJEKjBtDFhGuP9L7A3zRzZD8ZweAMnH2OFRjQysRyFL mFYf46A/eY7OeQW4wdVaAA7CVNaGzcDMoyl5cUZMqjmmXw0kCK0aSuDGMUEAgG1HZApb FJfXyIdpkb/ZbbNULOfoXA6ruXXa8QnQ/LMcliZ0SrY0E1N6rGDpcOTFI8a/HEhJwtTi nH8Azr/xdKaZjtxNC66hK/VE8tS/T89//79EU4PWOnxIh564NjKAU8lpIv+yYaDTjEVH d2wjri07VJhefeUF51b11RyHuWAYv2dvAAOx/UR3e4lqfL89lcM8MeXnr8CTu91BkqTT rLLw== X-Received: by 10.28.47.212 with SMTP id v203mr24292759wmv.37.1448371293864; Tue, 24 Nov 2015 05:21:33 -0800 (PST) Received: from localhost (ip-89-176-167-254.net.upcbroadband.cz. [89.176.167.254]) by smtp.gmail.com with ESMTPSA id pn6sm18264514wjb.15.2015.11.24.05.21.33 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 24 Nov 2015 05:21:33 -0800 (PST) From: Andreas Fenkart X-Google-Original-From: Andreas Fenkart To: u-boot@lists.denx.de Date: Tue, 24 Nov 2015 14:21:11 +0100 Message-Id: <1448371275-900-2-git-send-email-andreas.fenkart@dev.digitalstrom.org> X-Mailer: git-send-email 2.6.2 In-Reply-To: <1448371275-900-1-git-send-email-andreas.fenkart@dev.digitalstrom.org> References: <1448371275-900-1-git-send-email-andreas.fenkart@dev.digitalstrom.org> X-Mailman-Approved-At: Tue, 24 Nov 2015 14:30:29 +0100 Subject: [U-Boot] [PATCH 1/5] tools: env validate: pass values as 0-based array X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" passing argv/argc can produce off-by-one errors Signed-off-by: Andreas Fenkart --- common/env_flags.c | 14 +++++++------- include/env_flags.h | 2 +- tools/env/fw_env.c | 11 +++++++---- 3 files changed, 15 insertions(+), 12 deletions(-) diff --git a/common/env_flags.c b/common/env_flags.c index 985f92e..fb5ac40 100644 --- a/common/env_flags.c +++ b/common/env_flags.c @@ -359,21 +359,21 @@ int env_flags_validate_varaccess(const char *name, int check_mask) /* * Validate the parameters to "env set" directly */ -int env_flags_validate_env_set_params(int argc, char * const argv[]) +int env_flags_validate_env_set_params(char *name, char * const val[], int count) { - if ((argc >= 3) && argv[2] != NULL) { - enum env_flags_vartype type = env_flags_get_type(argv[1]); + if ((count >= 1) && val[0] != NULL) { + enum env_flags_vartype type = env_flags_get_type(name); /* * we don't currently check types that need more than * one argument */ - if (type != env_flags_vartype_string && argc > 3) { - printf("## Error: too many parameters for setting " - "\"%s\"\n", argv[1]); + if (type != env_flags_vartype_string && count > 1) { + printf("## Error: too many parameters for setting \"%s\"\n", + name); return -1; } - return env_flags_validate_type(argv[1], argv[2]); + return env_flags_validate_type(name, val[0]); } /* ok */ return 0; diff --git a/include/env_flags.h b/include/env_flags.h index 3ef6311..b153a71 100644 --- a/include/env_flags.h +++ b/include/env_flags.h @@ -121,7 +121,7 @@ int env_flags_validate_varaccess(const char *name, int check_mask); /* * Validate the parameters passed to "env set" for type compliance */ -int env_flags_validate_env_set_params(int argc, char * const argv[]); +int env_flags_validate_env_set_params(char *name, char *const val[], int count); #else /* !USE_HOSTCC */ diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 1173eea..bcf3756 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -481,8 +481,9 @@ int fw_setenv(int argc, char *argv[]) { int i, rc; size_t len; - char *name; + char *name, **valv; char *value = NULL; + int valc; if (argc < 2) { errno = EINVAL; @@ -513,13 +514,15 @@ int fw_setenv(int argc, char *argv[]) } name = argv[1]; + valv = argv + 2; + valc = argc - 2; - if (env_flags_validate_env_set_params(argc, argv) < 0) + if (env_flags_validate_env_set_params(name, valv, valc) < 0) return 1; len = 0; - for (i = 2; i < argc; ++i) { - char *val = argv[i]; + for (i = 0; i < valc; ++i) { + char *val = valv[i]; size_t val_len = strlen(val); if (value)