diff options
author | Eric Wong <normalperson@yhbt.net> | 2012-01-04 18:01:51 (JST) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2012-03-18 18:26:31 (JST) |
commit | 21418ec42a9a2de4c2c22eca7a1183b311914eca (patch) | |
tree | 76466eb627a017c57686530f0458699cb60d5684 /ui-shared.c | |
parent | 6a575b8900734a4640427416885e73c83af70736 (diff) | |
download | cgit-21418ec42a9a2de4c2c22eca7a1183b311914eca.zip cgit-21418ec42a9a2de4c2c22eca7a1183b311914eca.tar.gz |
segfault fix on some bogus requests
ctx.qry.head can be NULL in some cases due to bad requests
by weird bots. I managed to reproduce with:
PATH_INFO=/repo.git/shop.php QUERY_STRING=id=
Signed-off-by: Eric Wong <normalperson@yhbt.net>
Diffstat (limited to 'ui-shared.c')
-rw-r--r-- | ui-shared.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ui-shared.c b/ui-shared.c index 3e9282f..d7d75bf 100644 --- a/ui-shared.c +++ b/ui-shared.c | |||
@@ -294,7 +294,7 @@ void cgit_log_link(const char *name, const char *title, const char *class, | |||
294 | char *delim; | 294 | char *delim; |
295 | 295 | ||
296 | delim = repolink(title, class, "log", head, path); | 296 | delim = repolink(title, class, "log", head, path); |
297 | if (rev && strcmp(rev, ctx.qry.head)) { | 297 | if (rev && ctx.qry.head && strcmp(rev, ctx.qry.head)) { |
298 | html(delim); | 298 | html(delim); |
299 | html("id="); | 299 | html("id="); |
300 | html_url_arg(rev); | 300 | html_url_arg(rev); |
@@ -338,7 +338,7 @@ void cgit_commit_link(char *name, const char *title, const char *class, | |||
338 | char *delim; | 338 | char *delim; |
339 | 339 | ||
340 | delim = repolink(title, class, "commit", head, path); | 340 | delim = repolink(title, class, "commit", head, path); |
341 | if (rev && strcmp(rev, ctx.qry.head)) { | 341 | if (rev && ctx.qry.head && strcmp(rev, ctx.qry.head)) { |
342 | html(delim); | 342 | html(delim); |
343 | html("id="); | 343 | html("id="); |
344 | html_url_arg(rev); | 344 | html_url_arg(rev); |