changeset 56:43d088987482

Be nice and free some memory before exit
author mmj
date Wed, 28 Apr 2004 02:36:34 +1000
parents bf3ca0ce00ff
children 6fc67cdd9570
files src/mlmmj-bounce.c
diffstat 1 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/mlmmj-bounce.c	Wed Apr 28 02:07:42 2004 +1000
+++ b/src/mlmmj-bounce.c	Wed Apr 28 02:36:34 2004 +1000
@@ -83,18 +83,22 @@
 	filename = concatstr(2, listdir, "/subscribers");
 	if ((fd = open(filename, O_RDONLY)) < 0) {
 		log_error(LOG_ARGS, "Could not open '%s'", filename);
+		free(filename); free(bfilename);
 		exit(EXIT_FAILURE);
 	}
 	suboff = find_subscriber(fd, address);
-	if(suboff == -1)
+	if(suboff == -1) {
+		free(filename); free(bfilename);
 		exit(EXIT_SUCCESS); /* Not subbed, so exit silently */
+	}
+
 	free(filename);
 
-
 	/* TODO make sure the file we open below is not a symlink */
 	if ((fd = open(bfilename, O_WRONLY|O_APPEND|O_CREAT,
 			S_IRUSR|S_IWUSR)) < 0) {
 		log_error(LOG_ARGS, "Could not open '%s'", bfilename);
+		free(filename); free(bfilename);
 		exit(EXIT_FAILURE);
 	}
 	free(bfilename);