diff options
| author | Florian Pritz | 2010-02-01 17:55:37 +0100 |
|---|---|---|
| committer | Lars Hjemli | 2010-02-27 12:47:57 +0100 |
| commit | 8d946079c5f51ce934ff5edf6333962c6061d8a5 (patch) | |
| tree | 6d62be9c7f5e498f55a8c38f29b831fc3fc1d6b0 | |
| parent | CGIT 0.8.3.1 (diff) | |
| download | cgit-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.c | 4 |
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); |