diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-09-01 02:18:15 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-09-02 19:16:25 +0300 |
commit | d4a492f4ad0455710a7db8c589f4865810b0d0b7 (patch) | |
tree | 21e943eacb38558f68787b000291aacb83254418 /repository.h | |
parent | 86e16ed3a9fbf03bc8a5d4030177980193e30f57 (diff) |
submodule--helper: fix bad config API usage
Fix bad config API usage added in a452128a36c (submodule--helper:
introduce add-config subcommand, 2021-08-06). After
git_config_get_string() returns successfully we know the "char **dest"
will be non-NULL.
A coccinelle patch that transforms this turns up a couple of other
such issues, one in fetch-pack.c, and another in upload-pack.c:
@@
identifier F =~ "^(repo|git)_config_get_string(_tmp)?$";
identifier V;
@@
!F(..., &V)
- && (V)
But let's focus narrowly on submodule--helper for now, we can fix
those some other time.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Reviewed-by: Glen Choo <chooglen@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'repository.h')
0 files changed, 0 insertions, 0 deletions