From a0fb796c233f30cbbde56768f7073e954e02dfcb Mon Sep 17 00:00:00 2001 From: Sunny Kumar Date: Tue, 31 Jul 2018 19:03:05 +0530 Subject: geo-rep : fix few coverity issues in procdiggy.c This patch fixes BAD_FREE at line number 75,105 and UNUSED_VALUE at line number 108. CID : 1274068 scan detail at [1]. [1] https://scan6.coverity.com/reports.htm#v42401/p10714/fileInstanceId=84384726&defectInstanceId=25600457&mergedDefectId=727233 Change-Id: I8dff42546204bf78c178c9dcaba534888c5354e9 updates: bz#789278 Signed-off-by: Sunny Kumar --- geo-replication/src/procdiggy.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/geo-replication/src/procdiggy.c b/geo-replication/src/procdiggy.c index 35125d20ba8..4fb6a2405e4 100644 --- a/geo-replication/src/procdiggy.c +++ b/geo-replication/src/procdiggy.c @@ -72,8 +72,8 @@ pidinfo (pid_t pid, char **name) out: fclose (f); - if (pid == -1 && name && *name) - GF_FREE (name); + if (pid == -1 && name && *name) + GF_FREE (*name); if (pid == -2) fprintf (stderr, "out of memory\n"); return pid; @@ -104,8 +104,10 @@ prociter (int (*proch) (pid_t pid, pid_t ppid, char *tmpname, void *data), if (gf_string2int (de->d_name, &pid) != -1 && pid >= 0) { ppid = pidinfo (pid, &name); switch (ppid) { - case -1: continue; - case -2: ret = -1; break; + case -1: + continue; + case -2: + break; } ret = proch (pid, ppid, name, data); GF_FREE (name); -- cgit