lib/parser: Not recognize nul string as "%s" (6/6)

Hi,

Current match_token recognize "foo=" as "foo=%s". So this change the nul string does not recognize as "%s".

(Umm... this should be check by caller?)

 linux-2.6.0-test6-hirofumi/lib/parser.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletion(-)

diff -puN lib/parser.c~parser-zero-string lib/parser.c
@@ -45,7 +45,9 @@ static int match_one(char *s, char *p, s
 		args[argc].from = s;
 		switch (*p++) {
 		case 's':
-			if (len == -1 || len > strlen(s))
+			if (strlen(s) == 0)
+				return 0;
+			else if (len == -1 || len > strlen(s))
 				len = strlen(s);
 			args[argc].to = s + len;
 			break;

_

OGAWA Hirofumi <[email protected]>

On Tue, 07 Oct 2003 01:58:47 +0900 OGAWA Hirofumi <[email protected]> wrote:

| Hi,
| 
| Current match_token recognize "foo=" as "foo=%s". So this change the
| nul string does not recognize as "%s".
| 
| (Umm... this should be check by caller?)

I like the check here.

And thanks for doing all of these updates.

|  linux-2.6.0-test6-hirofumi/lib/parser.c |    4 +++-
|  1 files changed, 3 insertions(+), 1 deletion(-)

--
~Randy