diff options
| author | Russell Belfer <rb@github.com> | 2012-11-09 14:01:44 -0800 |
|---|---|---|
| committer | Russell Belfer <rb@github.com> | 2012-11-09 14:01:44 -0800 |
| commit | 757b406504021b3a73e52ce9f95d590d65c7dce5 (patch) | |
| tree | 8603837628030ae68b3bbc054f9bcf55eae53f43 /src/checkout.c | |
| parent | 0f3def715dc9af442f5f025c50a041c6319df1e8 (diff) | |
| download | libgit2-757b406504021b3a73e52ce9f95d590d65c7dce5.tar.gz | |
Fix warnings and valgrind issues
This fixes some various warnings that showed up in Travis and
a couple uses of uninitialized memory and one memory leak.
Diffstat (limited to 'src/checkout.c')
| -rw-r--r-- | src/checkout.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/checkout.c b/src/checkout.c index 8d164cfca..0d14e2625 100644 --- a/src/checkout.c +++ b/src/checkout.c @@ -432,8 +432,8 @@ static int checkout_get_actions( int error; git_diff_list *diff = data->diff; git_diff_delta *delta; - size_t i, *counts; - uint32_t *actions; + size_t i, *counts = NULL; + uint32_t *actions = NULL; git_tree *head = NULL; git_iterator *hiter = NULL; char *pfx = git_pathspec_prefix(&data->opts->paths); @@ -456,6 +456,7 @@ static int checkout_get_actions( goto fail; git__free(pfx); + pfx = NULL; *counts_ptr = counts = git__calloc(CHECKOUT_ACTION__MAX+1, sizeof(size_t)); *actions_ptr = actions = git__calloc(diff->deltas.length, sizeof(uint32_t)); @@ -509,6 +510,8 @@ static int checkout_get_actions( } git_iterator_free(hiter); + git_tree_free(head); + return 0; fail: @@ -518,6 +521,7 @@ fail: git__free(actions); git_iterator_free(hiter); + git_tree_free(head); git__free(pfx); return -1; |
