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 /commit-graph.c | |
| parent | submodule--helper: libify even more "die" paths for module_update() (diff) | |
| download | git-d4a492f4ad0455710a7db8c589f4865810b0d0b7.tar.gz git-d4a492f4ad0455710a7db8c589f4865810b0d0b7.zip | |
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 'commit-graph.c')
0 files changed, 0 insertions, 0 deletions