Skip to content

Commit

Permalink
adding LEAK SUMMARY
Browse files Browse the repository at this point in the history
  • Loading branch information
Ximaz committed May 2, 2024
1 parent 9bb648f commit fff1982
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,8 @@ jobs:
block="${line}"
elif [[ $(echo "${line}" | grep '^==.*== Open file descriptor .*: .*$') ]]; then
block="${line}"
elif [[ $(echo "${line}" | grep '^==.*== LEAK SUMMARY:.*$') ]]; then
block="${line}"
fi
done < ${{ env.VALGRIND_REPORTS }}
exit "${status}"
Expand Down
3 changes: 1 addition & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,8 @@ If you have write your unit tests correctly, and have the most coverage
possible, then the valgrind tests will make sure that the code you wrote, both
the tests and the actual implementations, are not leaking at all. Valgrind will
look for :
- unfree'd memory,
- memory leaks (including inside `fork()` children),
- unclosed file descriptor,
- bad memory management inside `fork()` children,
- invalid read/writes,
- conditionnal jumps or move depends on uninitialised value(s)

Expand Down

0 comments on commit fff1982

Please sign in to comment.