From ca227bf5f3d09e4e0e8db2c2931780855601912a Mon Sep 17 00:00:00 2001 From: Kristaps Dzonsons Date: Sun, 25 Dec 2011 19:31:25 +0000 Subject: First memory leak in mandocdb.c freeing "struct of". Also properly const-ify several strings. --- mandocdb.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'mandocdb.c') diff --git a/mandocdb.c b/mandocdb.c index 04495315..cf92d6dd 100644 --- a/mandocdb.c +++ b/mandocdb.c @@ -1,4 +1,4 @@ -/* $Id: mandocdb.c,v 1.40 2011/12/25 16:53:51 schwarze Exp $ */ +/* $Id: mandocdb.c,v 1.41 2011/12/25 19:31:25 kristaps Exp $ */ /* * Copyright (c) 2011 Kristaps Dzonsons * Copyright (c) 2011 Ingo Schwarze @@ -1452,7 +1452,8 @@ static void ofile_argbuild(int argc, char *argv[], struct of **of) { char buf[MAXPATHLEN]; - char *sec, *arch, *title, *p; + const char *sec, *arch, *title; + char *p; int i, src_form; struct of *nof; @@ -1756,6 +1757,7 @@ ofile_dirbuild(const char *dir, const char* psec, const char *parch, if (verb > 1) printf("%s: scheduling\n", buf); + if (NULL == *of) { *of = nof; (*of)->first = nof; @@ -1774,7 +1776,10 @@ ofile_free(struct of *of) { struct of *nof; - while (of) { + if (NULL != of) + of = of->first; + + while (NULL != of) { nof = of->next; free(of->fname); free(of->sec); -- cgit v1.2.3-56-ge451