| commit | 8ddf3ca74f40bebd79cedba5ba82ca9f2eb33860 | |
| author | Jeff King <peff@peff.net> | |
| Fri, 13 Mar 2015 04:42:25 +0000 (13 00:42 -0400) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Fri, 13 Mar 2015 06:21:16 +0000 (12 23:21 -0700) | ||
| tree | 892d6a13773d6edd93a7c65691cd17c9f791947f | treesnapshot (tar.gz zip) |
| parent | e172755b1eeb061c8fafb5ae8e4e9e72f68a0cdd | commitdiff |
upload-pack: do not check NULL return of lookup_unknown_object
We check whether the return value of lookup_unknown_object
is NULL, but some code paths dereference it before our
check. This turns out not to be capable of causing a
segfault, though. The lookup_unknown_object function will
never return NULL, since the whole point is to allocate an
object struct if it does not find an existing one. So the
code here is not wrong, it is just confusing. Let's just
drop the NULL check.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
We check whether the return value of lookup_unknown_object
is NULL, but some code paths dereference it before our
check. This turns out not to be capable of causing a
segfault, though. The lookup_unknown_object function will
never return NULL, since the whole point is to allocate an
object struct if it does not find an existing one. So the
code here is not wrong, it is just confusing. Let's just
drop the NULL check.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| upload-pack.c | diffblobblamehistory |