Commit a4b332e8775a2e4fedf24c143b10f087ddf53d02
2018-10-24T01:21:21
submodule: grab the error while loading from config Previously, an error in `git_config_next` would be mistaken as a successful load, because the previous call would have succeeded. Coverity saw the subsequent check for a completed iteration as dead, so let's make it useful again. CID 1391374
diff --git a/src/submodule.c b/src/submodule.c
index 06b7de5..2bfc468 100644
--- a/src/submodule.c
+++ b/src/submodule.c
@@ -181,7 +181,7 @@ static int load_submodule_names(git_strmap *out, git_repository *repo, git_confi
if ((error = git_config_iterator_glob_new(&iter, cfg, key)) < 0)
return error;
- while (git_config_next(&entry, iter) == 0) {
+ while ((error = git_config_next(&entry, iter)) == 0) {
const char *fdot, *ldot;
fdot = strchr(entry->name, '.');
ldot = strrchr(entry->name, '.');