"Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > The release window is a fine time to concentrate on bug fixes, Well, it certainly is better than random shiny new features or refactoring for the sake of refactoring, but finding and fixing a bug that has been with us since forever without anybody noticing is not all that ultra-urgent, either ;-) It is a good time to concentrate on finding and fixing regressions. > and Coverity > can help identify bugs. These are only a couple of fixes, Coverity reported > many more, but most of the reports seemed either bogus or harmless (such as > resource leaks in test helpers). Thanks. > > Johannes Schindelin (11): > mingw: avoid accessing uninitialized memory in `is_executable()` > fsmonitor: avoid memory leak in `fsm_settings__get_incompatible_msg()` > submodule--helper: avoid memory leak in `update_submodule()` > get_oid_with_context_1(): avoid use-after-free > submodule-config: avoid memory leak > pack-redundant: avoid using uninitialized memory > submodule--helper: avoid memory leak when fetching submodules > read_index_from(): avoid memory leak > pack-mtimes: avoid closing a bogus file descriptor > relative_url(): fix incorrect condition > bug_fl(): add missing `va_end()` call > > builtin/pack-redundant.c | 1 + > builtin/submodule--helper.c | 3 +++ > fsmonitor-settings.c | 8 ++++++-- > object-name.c | 6 ++++-- > pack-mtimes.c | 3 ++- > read-cache.c | 6 +++--- > remote.c | 2 +- > run-command.c | 2 +- > submodule-config.c | 8 ++++---- > usage.c | 1 + > 10 files changed, 26 insertions(+), 14 deletions(-) > > > base-commit: 8168d5e9c23ed44ae3d604f392320d66556453c9 > Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1264%2Fdscho%2Fcoverity-fixes-v1 > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1264/dscho/coverity-fixes-v1 > Pull-Request: https://github.com/gitgitgadget/git/pull/1264
