[PATCH] Stop crashing on win98

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Mon Jun 2 06:50:54 CDT 2008


---
 dlls/snmpapi/tests/util.c |   70 ++------------------------------------------
 1 files changed, 4 insertions(+), 66 deletions(-)

diff --git a/dlls/snmpapi/tests/util.c b/dlls/snmpapi/tests/util.c
index 428040d..5394eae 100644
--- a/dlls/snmpapi/tests/util.c
+++ b/dlls/snmpapi/tests/util.c
@@ -92,11 +92,6 @@ static void test_SnmpUtilOidToA(void)
         "1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1.1";
     static const char expect4[] = "-1";
 
-    ret = SnmpUtilOidToA(NULL);
-    ok(ret != NULL, "SnmpUtilOidToA failed\n");
-    ok(!strcmp(ret, expect0), "SnmpUtilOidToA failed got \n%s\n expected \n%s\n",
-       ret, expect1);
-
     ret = SnmpUtilOidToA(&oid0);
     ok(ret != NULL, "SnmpUtilOidToA failed\n");
     ok(!strcmp(ret, expect0), "SnmpUtilOidToA failed got \n%s\n expected \n%s\n",
@@ -195,25 +190,12 @@ static void test_SnmpUtilOidCpyFree(void)
     static UINT ids[] = { 1, 3, 6, 1, 4, 1, 311 };
     static AsnObjectIdentifier dst, src = { sizeof(ids) / sizeof(ids[0]), ids };
 
-    ret = SnmpUtilOidCpy(NULL, NULL);
-    ok(!ret, "SnmpUtilOidCpy succeeded\n");
-
-    memset(&dst, 1, sizeof(AsnObjectIdentifier));
-    ret = SnmpUtilOidCpy(&dst, NULL);
-    ok(ret, "SnmpUtilOidCpy failed\n");
-    ok(dst.idLength == 0, "SnmpUtilOidCpy failed\n");
-    ok(dst.ids == NULL, "SnmpUtilOidCpy failed\n");
-
-    ret = SnmpUtilOidCpy(NULL, &src);
-    ok(!ret, "SnmpUtilOidCpy succeeded\n");
-
     memset(&dst, 0, sizeof(AsnObjectIdentifier));
     ret = SnmpUtilOidCpy(&dst, &src);
     ok(ret, "SnmpUtilOidCpy failed\n");
     ok(src.idLength == dst.idLength, "SnmpUtilOidCpy failed\n");
     ok(!memcmp(src.ids, dst.ids, dst.idLength * sizeof(UINT)), "SnmpUtilOidCpy failed\n");
 
-    SnmpUtilOidFree(NULL);
     SnmpUtilOidFree(&dst);
     ok(dst.idLength == 0, "SnmpUtilOidFree failed\n");
     ok(dst.ids == NULL, "SnmpUtilOidFree failed\n");
@@ -292,24 +274,6 @@ static void test_SnmpUtilOidNCmp(void)
     static AsnObjectIdentifier oid1 = { 4, ids1 };
     static AsnObjectIdentifier oid2 = { 4, ids2 };
 
-    ret = SnmpUtilOidNCmp(NULL, NULL, 0);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
-    ret = SnmpUtilOidNCmp(NULL, NULL, 1);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
-    ret = SnmpUtilOidNCmp(&oid1, NULL, 0);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
-    ret = SnmpUtilOidNCmp(&oid1, NULL, 1);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
-    ret = SnmpUtilOidNCmp(NULL, &oid2, 0);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
-    ret = SnmpUtilOidNCmp(NULL, &oid2, 1);
-    ok(!ret, "SnmpUtilOidNCmp succeeded\n");
-
     ret = SnmpUtilOidNCmp(&oid1, &oid1, 0);
     ok(!ret, "SnmpUtilOidNCmp failed\n");
 
@@ -379,15 +343,6 @@ static void test_SnmpUtilOidAppend(void)
     oid1.idLength = 3;
     oid1.ids = ids1;
 
-    ret = SnmpUtilOidAppend(NULL, NULL);
-    ok(!ret, "SnmpUtilOidAppend succeeded\n");
-
-    ret = SnmpUtilOidAppend(&oid1, NULL);
-    ok(ret, "SnmpUtilOidAppend failed\n");
-
-    ret = SnmpUtilOidAppend(NULL, &oid2);
-    ok(!ret, "SnmpUtilOidAppend succeeded\n");
-
     ret = SnmpUtilOidAppend(&oid1, &oid2);
     ok(ret, "SnmpUtilOidAppend failed\n");
     ok(oid1.idLength == 6, "SnmpUtilOidAppend failed\n");
@@ -403,20 +358,6 @@ static void test_SnmpUtilVarBindCpyFree(void)
     static UINT ids[] = { 1, 3, 6, 1, 4, 1, 311 };
     static SnmpVarBind dst, src = { { 7, ids }, { ASN_INTEGER, { 1 } } };
 
-    ret = SnmpUtilVarBindCpy(NULL, NULL);
-    ok(!ret, "SnmpUtilVarBindCpy succeeded\n");
-
-    memset(&dst, 0, sizeof(SnmpVarBind));
-    ret = SnmpUtilVarBindCpy(&dst, NULL);
-    ok(ret, "SnmpUtilVarBindCpy failed\n");
-    ok(dst.name.idLength == 0, "SnmpUtilVarBindCpy failed\n");
-    ok(dst.name.ids == NULL, "SnmpUtilVarBindCpy failed\n");
-    ok(dst.value.asnType == ASN_NULL, "SnmpUtilVarBindCpy failed\n");
-    ok(dst.value.asnValue.number == 0, "SnmpUtilVarBindCpy failed\n");
-
-    ret = SnmpUtilVarBindCpy(NULL, &src);
-    ok(!ret, "SnmpUtilVarBindCpy succeeded\n");
-
     memset(&dst, 0, sizeof(SnmpVarBind));
     ret = SnmpUtilVarBindCpy(&dst, &src);
     ok(ret, "SnmpUtilVarBindCpy failed\n");
@@ -426,7 +367,6 @@ static void test_SnmpUtilVarBindCpyFree(void)
     ok(!memcmp(&src.value, &dst.value, sizeof(AsnObjectSyntax)),
        "SnmpUtilVarBindCpy failed\n");
 
-    SnmpUtilVarBindFree(NULL);
     SnmpUtilVarBindFree(&dst);
     ok(dst.name.idLength == 0, "SnmpUtilVarBindFree failed\n");
     ok(dst.name.ids == NULL, "SnmpUtilVarBindFree failed\n");
@@ -449,12 +389,6 @@ static void test_SnmpUtilVarBindListCpyFree(void)
     ok(!ret, "SnmpUtilVarBindListCpy succeeded\n");
     }
 
-    memset(&dst_list, 0xff, sizeof(SnmpVarBindList));
-    ret = SnmpUtilVarBindListCpy(&dst_list, NULL);
-    ok(ret, "SnmpUtilVarBindListCpy failed\n");
-    ok(dst_list.list == NULL, "SnmpUtilVarBindListCpy failed\n");
-    ok(dst_list.len == 0, "SnmpUtilVarBindListCpy failed\n");
-
     ret = SnmpUtilVarBindListCpy(&dst_list, &src_list);
     ok(ret, "SnmpUtilVarBindListCpy failed\n");
     ok(src_list.len == dst_list.len, "SnmpUtilVarBindListCpy failed\n");
@@ -503,8 +437,12 @@ START_TEST(util)
         test_SnmpUtilOctetsCmp();
 
     test_SnmpUtilOidCmp();
+
     test_SnmpUtilOidNCmp();
+
     test_SnmpUtilOidAppend();
+
     test_SnmpUtilVarBindCpyFree();
+
     test_SnmpUtilVarBindListCpyFree();
 }
-- 
1.5.4.1


--------------030909050408090304000901--




More information about the wine-patches mailing list