From f68502427091610bdbbe10a1eb4aabe0e5ac87d6 Mon Sep 17 00:00:00 2001
From: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
Date: Thu, 12 Apr 2018 14:31:24 +0000
Subject: [PATCH] Fix to bug error: non-void function return

This patch fix bug reported by

src/main.c:129:17: error: non-void function 'do_one_process'
    should return a value [-Wreturn-type]
                return;

Signed-off-by: Victor Rodriguez <victor.rodriguez.bahena@intel.com>
---
 src/main.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/main.c b/src/main.c
index eee053b..abc6610 100644
--- a/src/main.c
+++ b/src/main.c
@@ -45,6 +45,7 @@ struct process {
         int pid;
         char *name;
         uint64_t PSS_kb;
+        int valid;
 };
 
 struct process *allprocs=NULL;
@@ -125,8 +126,10 @@ struct process do_one_process(int pid)
                         fclose(file);
                 }
         }
-        if (process.name && searchkey && strstr(process.name, searchkey) == NULL)
-                return;
+        if (process.name && searchkey && strstr(process.name, searchkey) == NULL){
+                process.valid = 0;
+                return process;
+        }
 
         sprintf(filename, "/proc/%i/smaps", pid);
         file = fopen(filename, "r");
@@ -143,6 +146,7 @@ struct process do_one_process(int pid)
                 }
                 fclose(file);
         }
+        process.valid = 1;
         process.pid = pid;
         return process;
 }
@@ -192,7 +196,7 @@ int main(int argc, char **argv)
                 pid = strtoull(entry->d_name, NULL, 10);
                 if (pid){
                     process = do_one_process(pid);
-                    if (process.pid == pid){
+                    if (process.valid == 1 && process.pid == pid){
                         addnode(&process);
                         total_PSS += process.PSS_kb;
                         total_proc++;