From: Konrad Kuchciak <k.kuchciak@samsung.com>
Date: Tue, 25 Jun 2019 06:32:55 +0000 (+0200)
Subject: Add some sanity checks in data source modules
X-Git-Tag: submit/tizen/20190906.081624~8
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bc0f6dc2cd94c7441315df3b573d698d7e035311;p=platform%2Fcore%2Fsystem%2Fstability-monitor.git

Add some sanity checks in data source modules

Change-Id: Idd6da9f0b1cdb9e6a81e75c397571fdbc3b26978
---

diff --git a/src/data_sources/tsm_cpu.c b/src/data_sources/tsm_cpu.c
index 528a218..2e866fa 100644
--- a/src/data_sources/tsm_cpu.c
+++ b/src/data_sources/tsm_cpu.c
@@ -62,6 +62,11 @@ static int update(struct data_source *ds,
         return 0;
     }
 
+    if (ced->cpu_time < ced->cpu_time_last) {
+        _D_PROC(ds->process, "cpu_time < cpu_time_last, ignoring sample");
+        return 0;
+    }
+
     cpu_usage = (double) (ced->cpu_time - ced->cpu_time_last) / (ced->measure_time - ced->measure_time_last);
 
     ret = data_source_add_sample(ds, ST_DOUBLE, &cpu_usage, rd_global->v[0]);
diff --git a/src/data_sources/tsm_io.c b/src/data_sources/tsm_io.c
index 827add6..1f80786 100644
--- a/src/data_sources/tsm_io.c
+++ b/src/data_sources/tsm_io.c
@@ -58,6 +58,11 @@ static int update(struct data_source *ds,
         return 0;
     }
 
+    if (ied->io_bytes < ied->io_bytes_last) {
+        _D_PROC(ds->process, "io_bytes < io_bytes_last, ignoring sample");
+        return 0;
+    }
+
     io_rate = (double) (ied->io_bytes - ied->io_bytes_last) /
               (ied->measure_time - ied->measure_time_last);