[tor-commits] [tor/master] tor_vsscanf(): Don't return -1 if '%%' doesn't match.

[nickm at torproject.org]

commit b80728a115213abfaab6b3a124caac617ce5d253
Author: George Kadianakis <desnacked at riseup.net>
Date:   Tue Apr 3 16:20:24 2012 +0200

    tor_vsscanf(): Don't return -1 if '%%' doesn't match.
    
    tor_vsscanf() is supposed to return the current number of matches on
    match failure.
---
 changes/bug5558      |    3 +++
 src/common/util.c    |    2 +-
 src/test/test_util.c |    6 +++++-
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/changes/bug5558 b/changes/bug5558
new file mode 100644
index 0000000..90ab3a6
--- /dev/null
+++ b/changes/bug5558
@@ -0,0 +1,3 @@
+  o Minor bugfixes
+    - Enforce correct return behavior of tor_vsscanf(), when the '%%'
+      pattern is used. Fixes bug 5558.
diff --git a/src/common/util.c b/src/common/util.c
index 266368c..551ee17 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -2773,7 +2773,7 @@ tor_vsscanf(const char *buf, const char *pattern, va_list ap)
         ++n_matched;
       } else if (*pattern == '%') {
         if (*buf != '%')
-          return -1;
+          return n_matched;
         ++buf;
         ++pattern;
       } else {
diff --git a/src/test/test_util.c b/src/test/test_util.c
index 88f00e0..f2123c5 100644
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -1384,7 +1384,6 @@ test_util_sscanf(void)
   test_eq(-1, tor_sscanf("wrong", "%5c", s1)); /* %c cannot have a number. */
   test_eq(-1, tor_sscanf("hello", "%s", s1)); /* %s needs a number. */
   test_eq(-1, tor_sscanf("prettylongstring", "%999999s", s1));
-  test_eq(-1, tor_sscanf("We're the 99 monkeys", "We're the 99%%"));
 #if 0
   /* GCC thinks these two are illegal. */
   test_eq(-1, tor_sscanf("prettylongstring", "%0s", s1));
@@ -1468,6 +1467,11 @@ test_util_sscanf(void)
   /* Literal '%' (ie. '%%') */
   test_eq(1, tor_sscanf("99% fresh", "%3u%% fresh", &u1));
   test_eq(99, u1);
+  test_eq(0, tor_sscanf("99 fresh", "%% %3u %s", &u1, s1));
+  test_eq(1, tor_sscanf("99 fresh", "%3u%% %s", &u1, s1));
+  test_eq(2, tor_sscanf("99 fresh", "%3u %5s %%", &u1, s1));
+  test_eq(99, u1);
+  test_streq(s1, "fresh");
   test_eq(1, tor_sscanf("% boo", "%% %3s", s1));
   test_streq("boo", s1);