From: David Turner <dturner@twopensource.com>
Date: Wed, 18 Nov 2015 00:31:55 GMT
Subject: verify_pack ignores return value of verify_fn
Message-ID: <1447806715.5082.11.camel@twopensource.com>

In pack-check.c, line 129, a caller-supplied verification function is
called.  The function returns an int, but that return value is ignored.

The only caller of verify_pack is in builtin/fsck.c, whose verify_fn
*does* return a meaningful error code (which is then ignored).  If it
were not ignored, fsck might return a different error code (in the
unlikely event that a weird object gets into a pack and is somehow not 
totally corrupt enough to fail an earlier check).

I think we should probably have verify_pack return a non-zero result if
any call to verify_fn returns a non-zero result.  Any objections to
this?