about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLukas Fleischer <cgit@cryptocrack.de>2011-03-31 01:21:39 +0200
committerLars Hjemli <hjemli@gmail.com>2011-05-23 22:58:35 +0200
commit070e109c1413d28b54eb6123a9fd24ac98897554 (patch)
tree70f10262cdbc29cb9d083d24c03b1524a3d90ecd
parentRemove unused variable from cgit_diff_tree(). (diff)
downloadcgit-pink-070e109c1413d28b54eb6123a9fd24ac98897554.tar.gz
cgit-pink-070e109c1413d28b54eb6123a9fd24ac98897554.zip
Fix memory leak in http_parse_querystring().
Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r--html.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/html.c b/html.c
index 6f31097..a0f6db4 100644
--- a/html.c
+++ b/html.c
@@ -290,12 +290,12 @@ char *convert_query_hexchar(char *txt)
 
 int http_parse_querystring(const char *txt_, void (*fn)(const char *name, const char *value))
 {
-	char *t, *txt, *value = NULL, c;
+	char *o, *t, *txt, *value = NULL, c;
 
 	if (!txt_)
 		return 0;
 
-	t = txt = strdup(txt_);
+	o = t = txt = strdup(txt_);
 	if (t == NULL) {
 		printf("Out of memory\n");
 		exit(1);
@@ -318,5 +318,6 @@ int http_parse_querystring(const char *txt_, void (*fn)(const char *name, const
 	}
 	if (t!=txt)
 		(*fn)(txt, value);
+	free(o);
 	return 0;
 }