Patchwork [v2,2/8] fedfsd: don't return freed memory through **pathname parameter, ...

login
register
mail settings
Submitter Jim Meyering
Date Dec. 1, 2011, 12:27 p.m.
Message ID <1322742439-2171-3-git-send-email-jim@meyering.net>
Download mbox | patch
Permalink /patch/128690/
State Superseded
Headers show

Comments

Jim Meyering - Dec. 1, 2011, 12:27 p.m.
From: Jim Meyering <meyering@redhat.com>

even though upon error it will not be used.
* src/fedfsd/svc.c (fedfsd_pathwalk): On an error path, don't
set *pathname at all, and certainly not to a just-freed pointer.
Spotted by coverity.

Signed-off-by: Jim Meyering <meyering@redhat.com>
---
 src/fedfsd/svc.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

Patch

diff --git a/src/fedfsd/svc.c b/src/fedfsd/svc.c
index 132be70..12940c7 100644
--- a/src/fedfsd/svc.c
+++ b/src/fedfsd/svc.c
@@ -316,10 +316,12 @@  fedfsd_pathwalk(const FedFsPathName fpath, char **pathname)
 		xlog(D_CALL, "%s: Zero-component pathname", __func__);
 		strcat(result, "/");
 		retval = fedfsd_pathwalk_check_term(result);
-		if (retval != FEDFS_OK)
+		if (retval != FEDFS_OK) {
 			free(result);
+			return retval;
+		}
 		*pathname = result;
-		return retval;
+		return FEDFS_OK;
 	}

 	for (i = 0; i < fpath.FedFsPathName_len; i++) {