Workaround 'resource leak' error reported by cppcheck (tools, test)

* tests/test.c [GC_GCJ_SUPPORT && TEST_WITH_SYSTEM_MALLOC] (reverse):
Call GC_noop1 for malloc() result.
* tools/if_not_there.c (main): Move "f" local variable assignments
outside conditional expression.

diff --git a/tests/test.c b/tests/test.c
index e00ffd190bf69a86a011724a41358c55e8004da6..49c1337447564425dd6f43377c5638ecd78f6bd2 100644 (file)
--- a/tests/test.c
+++ b/tests/test.c
@@ -360,7 +360,7 @@ sexpr reverse1(sexpr x, sexpr y)
 sexpr reverse(sexpr x)
 {
 #   ifdef TEST_WITH_SYSTEM_MALLOC
-      malloc(100000);
+      GC_noop1(GC_HIDE_POINTER(malloc(100000)));
 #   endif
     return( reverse1(x, nil) );
 }

diff --git a/tools/if_not_there.c b/tools/if_not_there.c
index c0f095d058cae804efd53997471a70d4a7dff63f..652126d44da5ab2cbfcd034f68a8f9c45cf7155d 100644 (file)
--- a/tools/if_not_there.c
+++ b/tools/if_not_there.c
@@ -16,8 +16,14 @@ int main(int argc, char **argv)
     DIR * d;
 #endif /* __DJGPP__ */
     if (argc < 3) goto Usage;
-    if ((f = fopen(argv[1], "rb")) != 0
-        || (f = fopen(argv[1], "r")) != 0) {
+
+    f = fopen(argv[1], "rb");
+    if (f != NULL) {
+        fclose(f);
+        return(0);
+    }
+    f = fopen(argv[1], "r");
+    if (f != NULL) {
         fclose(f);
         return(0);
     }