ui-tree,ui-blame: bail from blame if blob is binary

This avoids piping binary blobs through the source-filter. Also prevent
robots from crawling it, since it's expensive.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/ui-tree.c b/ui-tree.c
index fbaaa32..4657c37 100644
--- a/ui-tree.c
+++ b/ui-tree.c
@@ -89,6 +89,7 @@ static void print_object(const struct object_id *oid, const char *path, const ch
         enum object_type type;
         char *buf;
         unsigned long size;
+        bool is_binary;
 
         type = oid_object_info(the_repository, oid, &size);
         if (type == OBJ_BAD) {
@@ -103,6 +104,7 @@ static void print_object(const struct object_id *oid, const char *path, const ch
                         "Error reading object %s", oid_to_hex(oid));
                 return;
         }
+        is_binary = buffer_is_binary(buf, size);
 
         cgit_set_title_from_path(path);
 
@@ -110,7 +112,7 @@ static void print_object(const struct object_id *oid, const char *path, const ch
         htmlf("blob: %s (", oid_to_hex(oid));
         cgit_plain_link("plain", NULL, NULL, ctx.qry.head,
                         rev, path);
-        if (ctx.repo->enable_blame) {
+        if (ctx.repo->enable_blame && !is_binary) {
                 html(") (");
                 cgit_blame_link("blame", NULL, NULL, ctx.qry.head,
                                 rev, path);
@@ -123,7 +125,7 @@ static void print_object(const struct object_id *oid, const char *path, const ch
                 return;
         }
 
-        if (buffer_is_binary(buf, size))
+        if (is_binary)
                 print_binary_buffer(buf, size);
         else
                 print_text_buffer(basename, buf, size);