From: Simon Wilkinson <sxw@your-file-system.com>
Date: Fri, 15 Feb 2013 16:10:18 +0000 (+0000)
Subject: rxgen: Fix memory leak
X-Git-Tag: upstream/1.8.0_pre1^2~1514
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=58cb6e60a8189e05cb5fd15172b09cbe6120fdc7;p=packages%2Fo%2Fopenafs.git

rxgen: Fix memory leak

Don't leak the definition structure when we hit the end of the file

Change-Id: Ief44bb3bdff274167760fda5bcc16903ef8c11ce
Reviewed-on: http://gerrit.openafs.org/9159
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/rxgen/rpc_parse.c b/src/rxgen/rpc_parse.c
index c8b94ca88..5aed967a1 100644
--- a/src/rxgen/rpc_parse.c
+++ b/src/rxgen/rpc_parse.c
@@ -184,7 +184,8 @@ get_definition(void)
 	def_const(defp);
 	break;
     case TOK_EOF:
-	return (NULL);
+	free(defp);
+	return NULL;
     case TOK_PACKAGE:
 	def_package(defp);
 	break;