about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorFlorian Pritz2010-02-01 17:55:37 +0100
committerLars Hjemli2010-02-27 12:47:57 +0100
commit8d946079c5f51ce934ff5edf6333962c6061d8a5 (patch)
tree6d62be9c7f5e498f55a8c38f29b831fc3fc1d6b0
parentCGIT 0.8.3.1 (diff)
downloadcgit-8d946079c5f51ce934ff5edf6333962c6061d8a5.tar.gz
cgit-8d946079c5f51ce934ff5edf6333962c6061d8a5.zip
ui-shared.c: fix segfault when repo is empty
PATH_INFO="/$REPONAME/commit/?id=1" QUERY_STRING="id=1" ./cgit.cgi
triggers segfault when the repository is empty and therefore
ctx.qry.head is unset

Signed-off-by: Florian Pritz <bluewind@xssn.at>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r--ui-shared.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/ui-shared.c b/ui-shared.c index 4049a2b..8a7cc32 100644 --- a/ui-shared.c +++ b/ui-shared.c
@@ -246,7 +246,7 @@ static void reporevlink(char *page, char *name, char *title, char *class,
246 char *delim; 246 char *delim;
247 247
248 delim = repolink(title, class, page, head, path); 248 delim = repolink(title, class, page, head, path);
249 if (rev && strcmp(rev, ctx.qry.head)) { 249 if (rev && ctx.qry.head != NULL && strcmp(rev, ctx.qry.head)) {
250 html(delim); 250 html(delim);
251 html("id="); 251 html("id=");
252 html_url_arg(rev); 252 html_url_arg(rev);
@@ -346,7 +346,7 @@ void cgit_diff_link(char *name, char *title, char *class, char *head,
346 char *delim; 346 char *delim;
347 347
348 delim = repolink(title, class, "diff", head, path); 348 delim = repolink(title, class, "diff", head, path);
349 if (new_rev && strcmp(new_rev, ctx.qry.head)) { 349 if (new_rev && ctx.qry.head != NULL && strcmp(new_rev, ctx.qry.head)) {
350 html(delim); 350 html(delim);
351 html("id="); 351 html("id=");
352 html_url_arg(new_rev); 352 html_url_arg(new_rev);