Since the value mask can no longer be set after the mode command, we can make sure that there is no junk after the command/mode parameter in the command line. Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx> --- CHANGES | 1 + tools/i2cset.c | 3 +++ 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/CHANGES b/CHANGES index 999ff26..1ffe18f 100644 --- a/CHANGES +++ b/CHANGES @@ -5,6 +5,7 @@ SVN i2c-dev.h: Make value arrays const for block write functions i2cset: Add support for SMBus and I2C block writes Remove obsolete means to specify value mask + More stringent parameter validation 3.0.3 (2010-12-12) Makefile: Let the environment set CC and CFLAGS diff --git a/tools/i2cset.c b/tools/i2cset.c index 7750c03..a45a829 100644 --- a/tools/i2cset.c +++ b/tools/i2cset.c @@ -240,6 +240,9 @@ int main(int argc, char *argv[]) block[len] = value; } goto dofile; + } else if (argc != flags + 6) { + fprintf(stderr, "Error: Too many arguments!\n"); + help(); } } -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html