diff options
author | John Keeping <john@keeping.me.uk> | 2013-04-07 23:06:23 (JST) |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2013-04-07 23:07:58 (JST) |
commit | 1c32e008c8fda46f812c38f46ae7515bcf8002ee (patch) | |
tree | 5dbda13c3fd12af82f7eaf739e6274680030d821 | |
parent | b60e6bff75719a5fb0df970bac3be6b2726cf73a (diff) | |
download | cgit-1c32e008c8fda46f812c38f46ae7515bcf8002ee.zip cgit-1c32e008c8fda46f812c38f46ae7515bcf8002ee.tar.gz |
ui-blob: don't segfault when no path is given
It it possible to inspect blobs by specifying only the SHA-1, and CGit
provides links to do so, for example if a tag points directly at a blob.
In this case the path_items structure is never used, but creating it
still causes strlen to be run on a null pointer. Fix this.
This error was introduced by commit c1633c6 (Update git to v1.7.6.5 -
2013-03-02).
Signed-off-by: John Keeping <john@keeping.me.uk>
-rw-r--r-- | ui-blob.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -80,7 +80,7 @@ void cgit_print_blob(const char *hex, char *path, const char *head) | |||
80 | struct commit *commit; | 80 | struct commit *commit; |
81 | struct pathspec_item path_items = { | 81 | struct pathspec_item path_items = { |
82 | .match = path, | 82 | .match = path, |
83 | .len = strlen(path) | 83 | .len = path ? strlen(path) : 0 |
84 | }; | 84 | }; |
85 | struct pathspec paths = { | 85 | struct pathspec paths = { |
86 | .nr = 1, | 86 | .nr = 1, |