]> git.ipfire.org Git - thirdparty/git.git/commit - diff-delta.c
diff-delta: set size out-parameter to 0 for NULL delta
authorJeff King <peff@peff.net>
Thu, 5 Sep 2019 22:53:37 +0000 (18:53 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 6 Sep 2019 18:03:39 +0000 (11:03 -0700)
commite4b369069e4a7630233a045784d0b1e2425b0a05
treeb8d944e9736c98e4dd6ca7a0c3e0a00bdafa0d6d
parent7140414d8bd7ed1a05b83bc34d0d0e76ef8b12bd
diff-delta: set size out-parameter to 0 for NULL delta

When we cannot generate a delta, we return NULL but leave delta_size
untouched. This is generally OK, as callers rely on NULL to decide if
the output is usable or not. But it can confuse compilers; in
particular, gcc 9.2.1 with "-flto -O3" complains in fast-import's
store_object() that delta_len may be used uninitialized.

Let's change the diff-delta code to set the size explicitly to 0 for a
NULL return. That silences the compiler and makes it easier to reason
about the result.

Reported-by: Stephan Beyer <s-beyer@gmx.net>
Helped-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff-delta.c