Fix some issues raise by coverity in the tests.
[openssl.git] / test / pbelutest.c
index c6ce586..84cb263 100644 (file)
 
 static int test_pbelu(void)
 {
-    int i, failed = 0, ok;
+    int i, failed = 0;
     int pbe_type, pbe_nid, last_type = -1, last_nid = -1;
 
     for (i = 0; EVP_PBE_get(&pbe_type, &pbe_nid, i) != 0; i++) {
         if (!TEST_true(EVP_PBE_find(pbe_type, pbe_nid, NULL, NULL, 0))) {
-            TEST_info("i=%d, pbe_type=%d, pbe_nid=%d", i, pbe_type, pbe_nid);
+            TEST_note("i=%d, pbe_type=%d, pbe_nid=%d", i, pbe_type, pbe_nid);
             failed = 1;
             break;
         }
@@ -33,20 +33,14 @@ static int test_pbelu(void)
 
     /* Error: print out whole table */
     for (i = 0; EVP_PBE_get(&pbe_type, &pbe_nid, i) != 0; i++) {
-        if (pbe_type > last_type)
-            ok = 0;
-        else if (pbe_type < last_type || pbe_nid < last_nid)
-            ok = 1;
-        else
-            ok = 0;
-        if (!ok)
-            failed = 1;
-        TEST_info("PBE type=%d %d (%s): %s\n", pbe_type, pbe_nid,
-                OBJ_nid2sn(pbe_nid), ok ? "ERROR" : "OK");
+        failed = pbe_type < last_type
+                 || (pbe_type == last_type && pbe_nid < last_nid);
+        TEST_note("PBE type=%d %d (%s): %s\n", pbe_type, pbe_nid,
+                  OBJ_nid2sn(pbe_nid), failed ? "ERROR" : "OK");
         last_type = pbe_type;
         last_nid = pbe_nid;
     }
-    return failed ? 0 : 1;
+    return 0;
 }
 
 void register_tests(void)