diff options
| author |  René Scharfe <l.s.r@web.de> | 2017-04-16 18:55:46 +0200 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2017-04-17 17:37:10 -0700 |
| commit | be686f03e0f4c4f14f1d4ae9b1b35836168a0a4b (patch) | |
| tree | 2c362df53918fca9ec5db1e3fdb535e2eccc9ea6 /t/t9604 | |
| parent | am: close stream on error, but not stdin (diff) | |
| download | tgif-be686f03e0f4c4f14f1d4ae9b1b35836168a0a4b.tar.xz | |
files_for_each_reflog_ent_reverse(): close stream and free strbuf on error
Exit the loop orderly through the cleanup code, instead of dashing out
with logfp still open and sb leaking.
Found with Cppcheck.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Reviewed-by: Jeff King <peff@peff.net>
Reviewed-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t9604')
0 files changed, 0 insertions, 0 deletions