about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLars Hjemli <hjemli@gmail.com>2009-08-17 09:05:13 +0200
committerLars Hjemli <hjemli@gmail.com>2009-08-17 09:26:17 +0200
commit435a1da8d1c43bff2f2ccd5649ea8510eec0b2af (patch)
treec07a9d096c99a70e78b017b5edccb1eaaffd0795
parentui-tag.c: do not segfault when id is missing from query-string (diff)
downloadcgit-pink-435a1da8d1c43bff2f2ccd5649ea8510eec0b2af.tar.gz
cgit-pink-435a1da8d1c43bff2f2ccd5649ea8510eec0b2af.zip
cgit.c: do not segfault on unexpected query-string format
The querystring_cb() function will be invoked with a NULL value when
the querystring contains a name not followed by a '='. Such a value
used to cause a segfault, which this patch fixes.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (limited to '')
-rw-r--r--cgit.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/cgit.c b/cgit.c
index 64d95f9..5301840 100644
--- a/cgit.c
+++ b/cgit.c
@@ -132,6 +132,9 @@ void config_cb(const char *name, const char *value)
 
 static void querystring_cb(const char *name, const char *value)
 {
+	if (!value)
+		value = "";
+
 	if (!strcmp(name,"r")) {
 		ctx.qry.repo = xstrdup(value);
 		ctx.repo = cgit_get_repoinfo(value);