]> granicus.if.org Git - git/commit
projects / git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ac8ce18) | patch
files_for_each_reflog_ent_reverse(): close stream and free strbuf on error
authorRené Scharfe <l.s.r@web.de>
Sun, 16 Apr 2017 16:55:46 +0000 (18:55 +0200)
committerJunio C Hamano <gitster@pobox.com>
Tue, 18 Apr 2017 00:37:10 +0000 (17:37 -0700)
commitbe686f03e0f4c4f14f1d4ae9b1b35836168a0a4b
tree2c362df53918fca9ec5db1e3fdb535e2eccc9ea6tree | snapshot
parentac8ce18d89acccaa7a66900adff75d4aeb6ec80bcommit | diff
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>
refs/files-backend.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.