cgit: prepare repo before error pages

[cgit.git] / ui-blob.c

diff --git a/ui-blob.c b/ui-blob.c
index 2f8bb7a006fc82ffe6f827f46bb3793e7d99e1fa..761e88673ff5360e39718ee1b94c53e67a0fde1e 100644 (file)
--- a/ui-blob.c
+++ b/ui-blob.c
@@ -13,7 +13,7 @@
 
 struct walk_tree_context {
        const char *match_path;
-       struct object_id matched_oid;
+       struct object_id *matched_oid;
        unsigned int found_path:1;
        unsigned int file_only:1;
 };
@@ -28,7 +28,7 @@ static int walk_tree(const unsigned char *sha1, struct strbuf *base,
        if (strncmp(base->buf, walk_tree_ctx->match_path, base->len)
                || strcmp(walk_tree_ctx->match_path + base->len, pathname))
                return READ_TREE_RECURSIVE;
-       hashcpy(walk_tree_ctx->matched_oid.hash, sha1);
+       hashcpy(walk_tree_ctx->matched_oid->hash, sha1);
        walk_tree_ctx->found_path = 1;
        return 0;
 }
@@ -38,7 +38,7 @@ int cgit_ref_path_exists(const char *path, const char *ref, int file_only)
        struct object_id oid;
        unsigned long size;
        struct pathspec_item path_items = {
-               .match = path,
+               .match = xstrdup(path),
                .len = strlen(path)
        };
        struct pathspec paths = {
@@ -47,16 +47,19 @@ int cgit_ref_path_exists(const char *path, const char *ref, int file_only)
        };
        struct walk_tree_context walk_tree_ctx = {
                .match_path = path,
-               .matched_oid = oid,
+               .matched_oid = &oid,
                .found_path = 0,
                .file_only = file_only
        };
 
        if (get_oid(ref, &oid))
-               return 0;
+               goto done;
        if (sha1_object_info(oid.hash, &size) != OBJ_COMMIT)
-               return 0;
-       read_tree_recursive(lookup_commit_reference(oid.hash)->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
+               goto done;
+       read_tree_recursive(lookup_commit_reference(&oid)->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
+
+done:
+       free(path_items.match);
        return walk_tree_ctx.found_path;
 }
 
@@ -77,7 +80,7 @@ int cgit_print_file(char *path, const char *head, int file_only)
        };
        struct walk_tree_context walk_tree_ctx = {
                .match_path = path,
-               .matched_oid = oid,
+               .matched_oid = &oid,
                .found_path = 0,
                .file_only = file_only
        };
@@ -86,7 +89,7 @@ int cgit_print_file(char *path, const char *head, int file_only)
                return -1;
        type = sha1_object_info(oid.hash, &size);
        if (type == OBJ_COMMIT) {
-               commit = lookup_commit_reference(oid.hash);
+               commit = lookup_commit_reference(&oid);
                read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
                if (!walk_tree_ctx.found_path)
                        return -1;
@@ -120,7 +123,7 @@ void cgit_print_blob(const char *hex, char *path, const char *head, int file_onl
        };
        struct walk_tree_context walk_tree_ctx = {
                .match_path = path,
-               .matched_oid = oid,
+               .matched_oid = &oid,
                .found_path = 0,
                .file_only = file_only
        };
@@ -142,7 +145,7 @@ void cgit_print_blob(const char *hex, char *path, const char *head, int file_onl
        type = sha1_object_info(oid.hash, &size);
 
        if ((!hex) && type == OBJ_COMMIT && path) {
-               commit = lookup_commit_reference(oid.hash);
+               commit = lookup_commit_reference(&oid);
                read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
                type = sha1_object_info(oid.hash, &size);
        }