[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 -----
- [Confuse-devel] address@hidden: [PATCH] confuse: correctly free all memory],
Martin Hedenfalk <=