diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-09-01 01:18:15 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-09-02 09:16:25 -0700 |
| commit | d4a492f4ad0455710a7db8c589f4865810b0d0b7 (patch) | |
| tree | 21e943eacb38558f68787b000291aacb83254418 /t/t0028-working-tree-encoding.sh | |
| parent | 86e16ed3a9fbf03bc8a5d4030177980193e30f57 (diff) | |
| download | git-d4a492f4ad0455710a7db8c589f4865810b0d0b7.tar.gz | |
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 't/t0028-working-tree-encoding.sh')
0 files changed, 0 insertions, 0 deletions