confuse-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Confuse-devel] address@hidden: [PATCH] confuse: correctly free all memo


From: Martin Hedenfalk
Subject: [Confuse-devel] address@hidden: [PATCH] confuse: correctly free all memory]
Date: Tue, 3 Jun 2014 13:33:58 +0200
User-agent: Mutt/1.5.23 (2014-03-12)

Aurelien, I took the liberty of forwarding your patch to the mailing list for
wider testing.

I'm looking for confirmation that this doesn't break anything.

        .martin

----- Forwarded message from Aurelien Jarno <address@hidden> -----

Date: Sat, 24 May 2014 16:41:02 +0200
From: Aurelien Jarno <address@hidden>
To: address@hidden
Cc: Aurelien Jarno <address@hidden>
Subject: [PATCH] confuse: correctly free all memory
X-Mailer: git-send-email 2.0.0.rc0

When calling cfg_free(), we also have to free the memory allocated by
the lexer to avoid memory leak.

Signed-off-by: Aurelien Jarno <address@hidden>
---
 src/confuse.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/confuse.c b/src/confuse.c
index 5e0dcee..b01a01a 100644
--- a/src/confuse.c
+++ b/src/confuse.c
@@ -50,6 +50,7 @@
 
 extern FILE *cfg_yyin;
 extern int cfg_yylex(cfg_t *cfg);
+extern void cfg_yylex_destroy();
 extern int cfg_lexer_include(cfg_t *cfg, const char *fname);
 extern void cfg_scan_string_begin(const char *buf);
 extern void cfg_scan_string_end(void);
@@ -1360,6 +1361,8 @@ DLLIMPORT void cfg_free(cfg_t *cfg)
     free(cfg->filename);
 
     free(cfg);
+
+    cfg_yylex_destroy();
 }
 
 DLLIMPORT int cfg_include(cfg_t *cfg, cfg_opt_t *opt, int argc,
-- 
2.0.0.rc0


----- End forwarded message -----



reply via email to

[Prev in Thread] Current Thread [Next in Thread]