about summary refs log tree commit diff
path: root/shared.c
diff options
context:
space:
mode:
authorLars Hjemli <hjemli@gmail.com>2009-02-01 19:29:47 +0100
committerLars Hjemli <hjemli@gmail.com>2009-02-01 19:29:47 +0100
commit5e447b1ed1aa751e8ec93dccf4df6fa4a7ffddb6 (patch)
tree823678b1c4a907cf0df50067c27fb5b752114df5 /shared.c
parentMerge branch 'lh/header' (diff)
parentshared.c: avoid SEGFAULT when checking for binary buffers (diff)
downloadcgit-pink-5e447b1ed1aa751e8ec93dccf4df6fa4a7ffddb6.tar.gz
cgit-pink-5e447b1ed1aa751e8ec93dccf4df6fa4a7ffddb6.zip
Merge branch 'lh/binary'
Diffstat (limited to '')
-rw-r--r--shared.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/shared.c b/shared.c
index 578a544..cce0af4 100644
--- a/shared.c
+++ b/shared.c
@@ -257,8 +257,8 @@ int filediff_cb(void *priv, mmbuffer_t *mb, int nbuf)
 }
 
 int cgit_diff_files(const unsigned char *old_sha1,
-		     const unsigned char *new_sha1,
-		     linediff_fn fn)
+		    const unsigned char *new_sha1, unsigned long *old_size,
+		    unsigned long *new_size, int *binary, linediff_fn fn)
 {
 	mmfile_t file1, file2;
 	xpparam_t diff_params;
@@ -268,6 +268,15 @@ int cgit_diff_files(const unsigned char *old_sha1,
 	if (!load_mmfile(&file1, old_sha1) || !load_mmfile(&file2, new_sha1))
 		return 1;
 
+	*old_size = file1.size;
+	*new_size = file2.size;
+
+	if ((file1.ptr && buffer_is_binary(file1.ptr, file1.size)) ||
+	    (file2.ptr && buffer_is_binary(file2.ptr, file2.size))) {
+		*binary = 1;
+		return 0;
+	}
+
 	memset(&diff_params, 0, sizeof(diff_params));
 	memset(&emit_params, 0, sizeof(emit_params));
 	memset(&emit_cb, 0, sizeof(emit_cb));