diff options
| author | Karol Lewandowski <k.lewandowsk@samsung.com> | 2020-04-02 13:32:36 +0200 |
|---|---|---|
| committer | Karol Lewandowski <k.lewandowsk@samsung.com> | 2020-04-02 14:26:16 +0200 |
| commit | 296b22a695bc3197a71d125abed926fe779ce4d8 (patch) | |
| tree | 9bd858167af9291956805763e09f257aa8ed9a06 | |
| parent | e7b5394c65076a7e91476446e33f4f52962f4eb7 (diff) | |
| download | crash-worker-296b22a695bc3197a71d125abed926fe779ce4d8.tar.gz crash-worker-296b22a695bc3197a71d125abed926fe779ce4d8.tar.bz2 crash-worker-296b22a695bc3197a71d125abed926fe779ce4d8.zip | |
dump_systemstate: Use compile-time parameter check in get_disk_used_percent()
Guard functions with assert as it's programming error to invoke
it with null path.
Change-Id: I3a312f90c0553c1dd9e9233551c0d42b89c260f7
| -rw-r--r-- | src/dump_systemstate/dump_systemstate.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/dump_systemstate/dump_systemstate.c b/src/dump_systemstate/dump_systemstate.c index 9b50fa5..149bfb0 100644 --- a/src/dump_systemstate/dump_systemstate.c +++ b/src/dump_systemstate/dump_systemstate.c @@ -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; |