dump_systemstate: Use compile-time parameter check in get_disk_used_percent() 46/229646/3
authorKarol Lewandowski <k.lewandowsk@samsung.com>
Thu, 2 Apr 2020 11:32:36 +0000 (13:32 +0200)
committerKarol Lewandowski <k.lewandowsk@samsung.com>
Thu, 2 Apr 2020 12:26:16 +0000 (14:26 +0200)
Guard functions with assert as it's programming error to invoke
it with null path.

Change-Id: I3a312f90c0553c1dd9e9233551c0d42b89c260f7

src/dump_systemstate/dump_systemstate.c

index 9b50fa5..149bfb0 100644 (file)
@@ -21,6 +21,7 @@
  * @brief   dump system states.
  */
 
+#include <assert.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -73,15 +74,13 @@ static void usage()
           );
 }
 
-/* get disk used percentage */
 static int get_disk_used_percent(const char *path)
 {
+       assert(path);
+
        struct statfs lstatfs;
        int percent;
 
-       if (!path)
-               return -1;
-
        if (statfs(path, &lstatfs) < 0)
                return -1;
        percent = (((lstatfs.f_blocks - lstatfs.f_bfree) * 1000) / (lstatfs.f_blocks)) + 9;