From: Dan McGee Date: Thu, 31 Mar 2011 01:24:34 GMT Subject: [PATCH 4/4] Unify pack header checking between index-pack and unpack-objects Message-ID: <1301534674-31981-4-git-send-email-dpmcgee@gmail.com> In-Reply-To: <1301534674-31981-1-git-send-email-dpmcgee@gmail.com> Both of these builtins share a lot of common code; taking steps to bring them closer together before a likely future commit to unify some of the shared code. Signed-off-by: Dan McGee --- builtin/unpack-objects.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/unpack-objects.c b/builtin/unpack-objects.c index a9e681e..a0dde7f 100644 --- a/builtin/unpack-objects.c +++ b/builtin/unpack-objects.c @@ -471,10 +471,10 @@ static uint32_t unpack_all(void) nr_objects = ntohl(hdr->hdr_entries); - if (ntohl(hdr->hdr_signature) != PACK_SIGNATURE) - die("bad pack file"); + if (hdr->hdr_signature != htonl(PACK_SIGNATURE)) + die("pack signature mismatch"); if (!pack_version_ok(hdr->hdr_version)) - die("unknown pack file version %"PRIu32, + die("pack version %"PRIu32" unsupported", ntohl(hdr->hdr_version)); use(sizeof(struct pack_header)); -- 1.7.4.2