Skip to content

Fix leaks in master#4636

Merged
pks-t merged 2 commits intolibgit2:masterfrom
tiennou:fix/leaks
Apr 20, 2018
Merged

Fix leaks in master#4636
pks-t merged 2 commits intolibgit2:masterfrom
tiennou:fix/leaks

Conversation

@tiennou
Copy link
Copy Markdown
Contributor

@tiennou tiennou commented Apr 20, 2018

The rest of the Valgrind + some Coverity fixes against master.

@tiennou tiennou mentioned this pull request Apr 20, 2018
tiennou added 2 commits April 20, 2018 19:37
CID:1383993, "In git_refspec__dwim_one: All paths that lead to this null pointer comparison already dereference the pointer earlier (CWE-476)"
Valgrind log:

==2711== 305 (48 direct, 257 indirect) bytes in 1 blocks are definitely lost in loss record 576 of 624
==2711==    at 0x4C2CC70: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2711==    by 0x5E079E: git__calloc (util.h:99)
==2711==    by 0x5E0D21: open_worktree_dir (worktree.c:134)
==2711==    by 0x5E0F23: git_worktree_lookup (worktree.c:176)
==2711==    by 0x5E1972: git_worktree_add (worktree.c:388)
==2711==    by 0x551F23: test_worktree_worktree__add_with_explicit_branch (worktree.c:292)
==2711==    by 0x45853E: clar_run_test (clar.c:222)
==2711==    by 0x4587E1: clar_run_suite (clar.c:286)
==2711==    by 0x458B04: clar_parse_args (clar.c:362)
==2711==    by 0x458CAB: clar_test_run (clar.c:428)
==2711==    by 0x45665C: main (main.c:24)
@pks-t pks-t merged commit 0ad2372 into libgit2:master Apr 20, 2018
@pks-t
Copy link
Copy Markdown
Member

pks-t commented Apr 20, 2018

And thanks again :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants