From 6adcaab70cdd6ced307a71945a1f80833ec9670f Mon Sep 17 00:00:00 2001 From: Edward Thomson Date: Wed, 8 Jan 2014 10:07:30 -0800 Subject: [PATCH] Handle git_buf's from users more liberally --- include/git2/blob.h | 2 +- src/blob.c | 2 ++ src/buffer.c | 8 +++++ src/buffer.h | 9 +++++ tests/filter/blob.c | 32 ++++++++++++++++++ .../2c/9a868cfdf8e270d0ec68164433376c68fb1789 | Bin 0 -> 169 bytes .../e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 | Bin 0 -> 15 bytes .../ef/0dcd356d77221e9c27f4f3928ad28e80b87ceb | Bin 0 -> 168 bytes .../resources/crlf/.gitted/refs/heads/master | Bin 41 -> 41 bytes 9 files changed, 52 insertions(+), 1 deletion(-) create mode 100644 tests/resources/crlf/.gitted/objects/2c/9a868cfdf8e270d0ec68164433376c68fb1789 create mode 100644 tests/resources/crlf/.gitted/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 create mode 100644 tests/resources/crlf/.gitted/objects/ef/0dcd356d77221e9c27f4f3928ad28e80b87ceb diff --git a/include/git2/blob.h b/include/git2/blob.h index 19ad4d949..6ba5e9f9c 100644 --- a/include/git2/blob.h +++ b/include/git2/blob.h @@ -84,7 +84,7 @@ GIT_EXTERN(git_repository *) git_blob_owner(const git_blob *blob); * time. * * @param blob pointer to the blob - * @return the pointer; NULL if the blob has no contents + * @return the pointer */ GIT_EXTERN(const void *) git_blob_rawcontent(const git_blob *blob); diff --git a/src/blob.c b/src/blob.c index ab344ae98..2e924f37f 100644 --- a/src/blob.c +++ b/src/blob.c @@ -347,6 +347,8 @@ int git_blob_filtered_content( assert(blob && path && out); + git_buf_sanitize(out); + if (check_for_binary_data && git_blob_is_binary(blob)) return 0; diff --git a/src/buffer.c b/src/buffer.c index 20682322e..3283c2d4f 100644 --- a/src/buffer.c +++ b/src/buffer.c @@ -100,6 +100,14 @@ void git_buf_free(git_buf *buf) git_buf_init(buf, 0); } +void git_buf_sanitize(git_buf *buf) +{ + if (buf->ptr == NULL) { + assert (buf->size == 0 && buf->asize == 0); + buf->ptr = git_buf__initbuf; + } +} + void git_buf_clear(git_buf *buf) { buf->size = 0; diff --git a/src/buffer.h b/src/buffer.h index c88af6fef..564a4f561 100644 --- a/src/buffer.h +++ b/src/buffer.h @@ -51,6 +51,15 @@ extern void git_buf_init(git_buf *buf, size_t initial_size); extern int git_buf_try_grow( git_buf *buf, size_t target_size, bool mark_oom, bool preserve_external); +/** + * Sanitizes git_buf structures provided from user input. Users of the + * library, when providing git_buf's, may wish to provide a NULL ptr for + * ease of handling. The buffer routines, however, expect a non-NULL ptr + * always. This helper method simply handles NULL input, converting to a + * git_buf__initbuf. + */ +extern void git_buf_sanitize(git_buf *buf); + extern void git_buf_swap(git_buf *buf_a, git_buf *buf_b); extern char *git_buf_detach(git_buf *buf); extern void git_buf_attach(git_buf *buf, char *ptr, size_t asize); diff --git a/tests/filter/blob.c b/tests/filter/blob.c index 9600a9779..8dce6470a 100644 --- a/tests/filter/blob.c +++ b/tests/filter/blob.c @@ -47,6 +47,38 @@ void test_filter_blob__all_crlf(void) git_blob_free(blob); } +void test_filter_blob__sanitizes(void) +{ + git_blob *blob; + git_buf buf; + + cl_git_pass(git_revparse_single( + (git_object **)&blob, g_repo, "e69de29")); /* zero-byte */ + + cl_assert_equal_i(0, git_blob_rawsize(blob)); + cl_assert_equal_s("", git_blob_rawcontent(blob)); + + memset(&buf, 0, sizeof(git_buf)); + cl_git_pass(git_blob_filtered_content(&buf, blob, "file.bin", 1)); + cl_assert_equal_sz(0, buf.size); + cl_assert_equal_s("", buf.ptr); + git_buf_free(&buf); + + memset(&buf, 0, sizeof(git_buf)); + cl_git_pass(git_blob_filtered_content(&buf, blob, "file.crlf", 1)); + cl_assert_equal_sz(0, buf.size); + cl_assert_equal_s("", buf.ptr); + git_buf_free(&buf); + + memset(&buf, 0, sizeof(git_buf)); + cl_git_pass(git_blob_filtered_content(&buf, blob, "file.lf", 1)); + cl_assert_equal_sz(0, buf.size); + cl_assert_equal_s("", buf.ptr); + git_buf_free(&buf); + + git_blob_free(blob); +} + void test_filter_blob__ident(void) { git_oid id; diff --git a/tests/resources/crlf/.gitted/objects/2c/9a868cfdf8e270d0ec68164433376c68fb1789 b/tests/resources/crlf/.gitted/objects/2c/9a868cfdf8e270d0ec68164433376c68fb1789 new file mode 100644 index 0000000000000000000000000000000000000000..218e9d192796e89b754f393ddaad23945938d501 GIT binary patch literal 169 zcmV;a09OBa0i}+?4FVw$0DV)%HlTq81QKKPVFxzAExYj&5rY_`)qB?9Z!(X`P^)zw zfX=T*X9gBss93)(As`w~gh8CxgfSP49Q70f@ucp#^KF1Z$zsWH_FjBI7c4O)?NKPZ zZ6xcXBbroxj!WxsDzDrNoG-0D+6{KR*dHD0Ouaptj19HA2V-M04ueUs)=_J<*kyF^ X9|?6V1wPyxOz*)U^Oe;X*tbw7(hN$C literal 0 HcmV?d00001 diff --git a/tests/resources/crlf/.gitted/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 b/tests/resources/crlf/.gitted/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 new file mode 100644 index 0000000000000000000000000000000000000000..711223894375fe1186ac5bfffdc48fb1fa1e65cc GIT binary patch literal 15 Wcmb7w`4FhFfcPQQAo_m(M>MONn=>K=*2`E*?k&9zm7iJ@?go{ zSl-AqBsD-4mD39@p8t{_ytH=vf;7?DVFI`Ete`4#^NUg;#_)e#YA?C}($;qicZD)d zJyI%~-r0_<2xtz=J%-H@AB{Vww1*WdM!Z_}exXYbR7q88QNC_cWl1W-v$>CEZ@PHR W*}GeFU3t`vLr;2wCjtNgYE5lVp-lz= literal 0 HcmV?d00001 diff --git a/tests/resources/crlf/.gitted/refs/heads/master b/tests/resources/crlf/.gitted/refs/heads/master index a2dbe0c2d074941d11b4a679e6b3db3139de2eed..cfdaaf37b2f34a17bf25b42e005f823da5a012fa 100644 GIT binary patch literal 41 ucmXp?woJ4zvq(-$NwY{bGB-#uNKH1gFf=nUF*Y_fHv_WMk_^o)EV%&g`3kxK literal 41 ucmV~$$q@h`2n4Xd)yT1e<=_tVUxJC~)EK1;7=(Hln|DKp