diff options
| author | Carlos Martín Nieto <cmn@dwim.me> | 2015-06-09 16:53:07 +0200 |
|---|---|---|
| committer | Carlos Martín Nieto <cmn@dwim.me> | 2015-06-09 16:53:07 +0200 |
| commit | 02980bdca1c3398396466adff993746cac34fc08 (patch) | |
| tree | d39d2f49e7f256ae00735a47f23320e37426b5f6 /src | |
| parent | 81be2f467ce679e5e1a4b650ebdfc15ce3a9deb7 (diff) | |
| download | libgit2-02980bdca1c3398396466adff993746cac34fc08.tar.gz | |
Initialize a few variables
Coverity complains about the git_rawobj ones because we use a loop in
which we keep remembering the old version, and we end up copying our
object as the base, so we want to have the data pointer be NULL.
Diffstat (limited to 'src')
| -rw-r--r-- | src/indexer.c | 2 | ||||
| -rw-r--r-- | src/odb_pack.c | 2 | ||||
| -rw-r--r-- | src/rebase.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/src/indexer.c b/src/indexer.c index e39345c71..ef2ac3cba 100644 --- a/src/indexer.c +++ b/src/indexer.c @@ -822,7 +822,7 @@ static int resolve_deltas(git_indexer *idx, git_transfer_progress *stats) progressed = 0; non_null = 0; git_vector_foreach(&idx->deltas, i, delta) { - git_rawobj obj; + git_rawobj obj = {NULL}; if (!delta) continue; diff --git a/src/odb_pack.c b/src/odb_pack.c index 1757cf920..735158d96 100644 --- a/src/odb_pack.c +++ b/src/odb_pack.c @@ -383,7 +383,7 @@ static int pack_backend__read_internal( git_odb_backend *backend, const git_oid *oid) { struct git_pack_entry e; - git_rawobj raw; + git_rawobj raw = {NULL}; int error; if ((error = pack_entry_find(&e, (struct pack_backend *)backend, oid)) < 0 || diff --git a/src/rebase.c b/src/rebase.c index b636e7951..8da7b4f7f 100644 --- a/src/rebase.c +++ b/src/rebase.c @@ -512,7 +512,7 @@ static int rebase_ensure_not_dirty( git_tree *head = NULL; git_index *index = NULL; git_diff *diff = NULL; - int error; + int error = 0; if (check_index) { if ((error = git_repository_head_tree(&head, repo)) < 0 || |