On Thu, 2017-06-01 at 10:45 -0400, Steve Dickson wrote: > Hello, > > On 05/22/2017 11:50 AM, Justin Mitchell wrote: > > Add function to cleanup and free the loaded config > > > > Signed-off-by: Justin Mitchell <jumitche@xxxxxxxxx> > > > > --- > > support/include/conffile.h | 1 + > > support/nfs/conffile.c | 51 +++++++++++++++++++++++++++++++++++++++------- > > 2 files changed, 45 insertions(+), 7 deletions(-) > > > > diff --git a/support/include/conffile.h b/support/include/conffile.h > > index 20b1a32..2d11a52 100644 > > --- a/support/include/conffile.h > > +++ b/support/include/conffile.h > > @@ -60,6 +60,7 @@ extern _Bool conf_get_bool(char *, char *, _Bool); > > extern char *conf_get_str(char *, char *); > > extern char *conf_get_section(char *, char *, char *); > > extern void conf_init(const char *); > > +extern void conf_cleanup(void); > I don't see this being called any where? How is it being used? > > steved. It is not directly referenced yet, for completeness all of the programs that use the conf file should call it as part of their shutdown, but of course thats not vital when your process is exiting anyway. it will however be more important for a library using the same code, and for anything that is going to re-load its config or handle multiple files. It also gets used in my memory leak testing. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
