Changeset 58153 in vbox

Timestamp:

Oct 9, 2015 1:42:14 PM (10 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

103260

Message:

cppcheck warnings

Location:

trunk/src/VBox/ValidationKit

Files:

• bootsectors/VBoxBs2Linker.cpp (modified) (2 diffs)
• snippets/alloc-1.c (modified) (2 diffs)
• snippets/time-1.c (modified) (2 diffs)

trunk/src/VBox/ValidationKit/bootsectors/VBoxBs2Linker.cpp

@@ -62 +62 @@
                 {
                     fprintf(stderr, "syntax errro: Unknown options '%s'\n", pszOpt);
+                    free(papszInputs);
                     return 2;
                 }
@@ -206 +207 @@
     }
 
+    fclose(pOutput);
     return rcExit;
 }

trunk/src/VBox/ValidationKit/snippets/alloc-1.c

@@ -53 +53 @@
     if (cb / (1024 * 1024) != cMBs)
         cb = ~(unsigned long)0 / (1024 * 1024) * (1024 * 1024);
-    printf("alloc-1: allocating %lu MB (%lu bytes)\n", cb/1024/1024, cb);
+    printf("alloc-1: allocating %ld MB (%lu bytes)\n", cb/1024/1024, cb);
 
     /*
@@ -69 +69 @@
         if (!pb)
         {
-            printf("\nalloc-1: calloc failed, cbDone=%lu MB (%lu bytes)\n",
-                   cbDone/1024/1024/1024, cbDone);
+            printf("\nalloc-1: calloc failed, cbDone=%ld MB (%lu bytes)\n",
+                   cbDone/1024/1024, cbDone);
             return 1;
         }

trunk/src/VBox/ValidationKit/snippets/time-1.c

@@ -72 +72 @@
         {
 #ifdef USE_CLOCK_MONOTONIC
-            printf("tv_nsec in the past: %ld.%09d < %ld.%09d - %u nsec\n",
+            printf("tv_nsec in the past: %ld.%09u < %ld.%09u - %d nsec\n",
                    (long)pCur->tv_sec,  (unsigned)pCur->tv_nsec,
                    (long)pPrev->tv_sec, (unsigned)pPrev->tv_nsec,
                    (unsigned)pPrev->tv_nsec - (unsigned)pCur->tv_nsec);
 #else
-            printf("tv_usec in the past: %ld.%06d < %ld.%06d - %u usec\n",
+            printf("tv_usec in the past: %ld.%06u < %ld.%06u - %d usec\n",
                    (long)pCur->tv_sec,  (unsigned)pCur->tv_usec,
                    (long)pPrev->tv_sec, (unsigned)pPrev->tv_usec,
@@ -89 +89 @@
         {
 #ifdef USE_CLOCK_MONOTONIC
-            printf("tv_sec  in the past: %ld.%09d < %ld.%09d\n",
+            printf("tv_sec  in the past: %ld.%09u < %ld.%09u\n",
                    (long)pCur->tv_sec,  (unsigned)pCur->tv_nsec,
                    (long)pPrev->tv_sec, (unsigned)pPrev->tv_nsec);
 #else
-            printf("tv_sec  in the past: %ld.%06d < %ld.%06d\n",
+            printf("tv_sec  in the past: %ld.%06u < %ld.%06u\n",
                    (long)pCur->tv_sec,  (unsigned)pCur->tv_usec,
                    (long)pPrev->tv_sec, (unsigned)pPrev->tv_usec);