Juan Lang : snmpapi: Test and correct SnmpUtilOidNCmp when count is less than Oid length.

Alexandre Julliard julliard at winehq.org
Mon May 19 16:25:43 CDT 2008


Module: wine
Branch: master
Commit: 3108e3605cbf9a0cd0becaeeab9d976f3a7f298c
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=3108e3605cbf9a0cd0becaeeab9d976f3a7f298c

Author: Juan Lang <juan.lang at gmail.com>
Date:   Fri May 16 16:09:34 2008 -0700

snmpapi: Test and correct SnmpUtilOidNCmp when count is less than Oid length.

---

 dlls/snmpapi/main.c       |    1 +
 dlls/snmpapi/tests/util.c |    5 +++++
 2 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/dlls/snmpapi/main.c b/dlls/snmpapi/main.c
index 0e60d88..9e14a19 100644
--- a/dlls/snmpapi/main.c
+++ b/dlls/snmpapi/main.c
@@ -356,6 +356,7 @@ INT WINAPI SnmpUtilOidNCmp(AsnObjectIdentifier *oid1, AsnObjectIdentifier *oid2,
         if (oid1->ids[i] > oid2->ids[i]) return 1;
         if (oid1->ids[i] < oid2->ids[i]) return -1;
     }
+    if (i == count) return 0;
     if (oid1->idLength < oid2->idLength) return -1;
     if (oid1->idLength > oid2->idLength) return 1;
     return 0;
diff --git a/dlls/snmpapi/tests/util.c b/dlls/snmpapi/tests/util.c
index 3d513bd..428040d 100644
--- a/dlls/snmpapi/tests/util.c
+++ b/dlls/snmpapi/tests/util.c
@@ -330,6 +330,11 @@ static void test_SnmpUtilOidNCmp(void)
     ok(ret == 1, "SnmpUtilOidNCmp failed: %d\n", ret);
     ret = SnmpUtilOidNCmp(&oid1, &oid2, 4);
     ok(ret == -1, "SnmpUtilOidNCmp failed: %d\n", ret);
+
+    ret = SnmpUtilOidNCmp(&oid1, &oid2, 2);
+    ok(!ret, "SnmpUtilOidNCmp failed: %d\n", ret);
+    ret = SnmpUtilOidNCmp(&oid2, &oid1, 2);
+    ok(!ret, "SnmpUtilOidNCmp failed: %d\n", ret);
 }
 
 static void test_SnmpUtilOidCmp(void)




More information about the wine-cvs mailing list