From 0c0bf8519aab878acb864d84185bd2395b1e3d5b Mon Sep 17 00:00:00 2001 From: David Gibson Date: Wed, 1 Jan 2014 23:23:54 +1100 Subject: Fix memory leak in srcpos_verror() Since dtc runs are short, we don't care that much about memory leaks. Still, leaking the source position string every time we print an error messages is pretty nasty. Fix it. Signed-off-by: David Gibson --- srcpos.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/srcpos.c b/srcpos.c index 48059aa..6705fed 100644 --- a/srcpos.c +++ b/srcpos.c @@ -293,13 +293,15 @@ srcpos_string(struct srcpos *pos) void srcpos_verror(struct srcpos *pos, const char *fmt, va_list va) { - const char *srcstr; + char *srcstr; srcstr = srcpos_string(pos); fprintf(stderr, "Error: %s ", srcstr); vfprintf(stderr, fmt, va); fprintf(stderr, "\n"); + + free(srcstr); } void -- cgit v1.2.1