These patches implement some defensive programming to address complaints some static analyzers might have. Johannes Schindelin (14): revision: defensive programming get_parent(): defensive programming fetch-pack: defensive programming unparse_commit(): defensive programming verify_commit_graph(): defensive programming stash: defensive programming stash: defensive programming push: defensive programming fetch: defensive programming describe: defensive programming inherit_tracking(): defensive programming submodule: check return value of `submodule_from_path()` test-tool repository: check return value of `lookup_commit()` shallow: handle missing shallow commits gracefully branch.c | 2 ++ builtin/describe.c | 2 ++ builtin/fetch.c | 3 ++- builtin/push.c | 2 +- builtin/stash.c | 7 ++++++- builtin/submodule--helper.c | 3 +++ commit-graph.c | 5 +++++ commit.c | 2 +- fetch-pack.c | 2 +- object-name.c | 2 +- revision.c | 3 +++ shallow.c | 3 ++- t/helper/test-repository.c | 2 ++ 13 files changed, 31 insertions(+), 7 deletions(-) base-commit: 683c54c999c301c2cd6f715c411407c413b1d84e Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1890%2Fdscho%2Fdefensive-programming-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1890/dscho/defensive-programming-v1 Pull-Request: https://github.com/gitgitgadget/git/pull/1890 -- gitgitgadget
