git.99rst.org / git.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ac8ce18)
files_for_each_reflog_ent_reverse(): close stream and free strbuf on error
authorRené Scharfe <redacted>
Sun, 16 Apr 2017 16:55:46 +0000 (18:55 +0200)
committerJunio C Hamano <redacted>
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 <redacted>
Reviewed-by: Jeff King <redacted>
Reviewed-by: Michael Haggerty <redacted>
Signed-off-by: Junio C Hamano <redacted>
refs/files-backend.c diff | blob | history
git clone https://git.99rst.org/PROJECT