From d3b8a05d229a80100f40fca4dfdcd820313fcea8 Mon Sep 17 00:00:00 2001 From: Marcio Barbosa Date: Tue, 28 Jun 2016 12:48:06 -0300 Subject: [PATCH] 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 Reviewed-by: Benjamin Kaduk --- src/venus/fs.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/venus/fs.c b/src/venus/fs.c index caf7ce35fa..a0dfa9a26f 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; }