From cc22b795fb5fee72bd567eec5d33a11e8b989086 Mon Sep 17 00:00:00 2001 From: Wolfgang Denk Date: Tue, 8 Feb 2011 16:56:05 +0100 Subject: [PATCH] itest: fix result of string compares The implementation of the string compare function of the "itest" command was weird, as only the length of the shortest argument was included in the compare, with the result that something like "itest.s abd == abddef" would return TRUE. Fix this. Signed-off-by: Wolfgang Denk Acked-by: Detlev Zundel --- common/cmd_itest.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/common/cmd_itest.c b/common/cmd_itest.c index fa6a0c301..2a238a43e 100644 --- a/common/cmd_itest.c +++ b/common/cmd_itest.c @@ -94,16 +94,13 @@ static char * evalstr(char *s) static int stringcomp(char *s, char *t, int op) { - int n, p; + int p; char *l, *r; l = evalstr(s); r = evalstr(t); - /* we'll do a compare based on the length of the shortest string */ - n = min(strlen(l), strlen(r)); - - p = strncmp(l, r, n); + p = strcmp(l, r); switch (op) { case EQ: return (p == 0); case NE: return (p != 0);