From: Marcio Barbosa <mbarbosa@sinenomine.net>
Date: Thu, 3 Mar 2016 21:23:28 +0000 (-0300)
Subject: afs: fix memory leak
X-Git-Tag: upstream/1.8.0_pre1^2~98
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=bc123573539084ffc5a16ef1efaaaced5b2be202;p=packages%2Fo%2Fopenafs.git

afs: fix memory leak

An error code is returned by afs_ProcessOpCreate if
this function can not allocate memory for ttargetName.

This function should release the memory previously
allocated for tname and decrement the reference count
of tdp as well.

Change-Id: Ic771b1d57080df6ee562a7327762030afdd5b08c
Reviewed-on: https://gerrit.openafs.org/12208
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/afs/afs_disconnected.c b/src/afs/afs_disconnected.c
index a83d076de..692dcbcd3 100644
--- a/src/afs/afs_disconnected.c
+++ b/src/afs/afs_disconnected.c
@@ -702,7 +702,8 @@ afs_ProcessOpCreate(struct vcache *avc, struct vrequest *areq,
 	ttargetName = afs_osi_Alloc(tlen);
 	if (!ttargetName) {
 	    afs_PutDCache(tdc);
-	    return ENOMEM;
+	    code = ENOMEM;
+	    goto end;
 	}
 	ObtainReadLock(&tdc->lock);
 	tfile = afs_CFileOpen(&tdc->f.inode);