From: Marcio Barbosa <mbarbosa@sinenomine.net>
Date: Tue, 28 Jun 2016 15:48:06 +0000 (-0300)
Subject: venus: fix memory leak
X-Git-Tag: upstream/1.8.0_pre1^2~77
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=d3b8a05d229a80100f40fca4dfdcd820313fcea8;p=packages%2Fo%2Fopenafs.git

venus: fix memory leak

The fs getserverprefs command displays preference
ranks for file / volume location server machine
interfaces. In order to get the complete set of
preference ranks, the VIOC_GETSPREFS system call
might have to be called several times. If so, the
memory previously allocated should be released.

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

diff --git a/src/venus/fs.c b/src/venus/fs.c
index caf7ce35f..a0dfa9a26 100644
--- a/src/venus/fs.c
+++ b/src/venus/fs.c
@@ -3351,10 +3351,10 @@ GetPrefCmd(struct cmd_syndesc *as, void *arock)
 	}
 
 	in->offset = out->next_offset;
-    } while (out->next_offset > 0);
-
-    if (blob.out != space)
-	free(blob.out);
+	if (blob.out != space) {
+	    free(blob.out);
+	}
+    } while (in->offset > 0);
 
     return 0;
 }