Signed-off-by: Sun He <sunheehnus@xxxxxxxxx> --- bundle.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bundle.c b/bundle.c index e99065c..7809fbb 100644 --- a/bundle.c +++ b/bundle.c @@ -19,7 +19,7 @@ static void add_to_ref_list(const unsigned char *sha1, const char *name, list->list = xrealloc(list->list, list->alloc * sizeof(list->list[0])); } - memcpy(list->list[list->nr].sha1, sha1, 20); + hashcpy(list->list[list->nr].sha1, sha1); list->list[list->nr].name = xstrdup(name); list->nr++; } -- 1.7.1 > See if you can find other places where hashcpy() should be used instead of memcpy() grep.c:grep_source_init() reflog-walk.c:read_one_reflog() ppc/sha1.c:ppc_SHA1_Final() refs.c:resolve_gitlink_packed_ref() We can find those by the shell command: $ find . | xargs grep "memcpy\(.*20.*\) Cheers, He Sun -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html