Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Automated memory leaks testing #995

Closed
even1024 opened this issue Jan 18, 2023 · 0 comments · Fixed by #1038
Closed

Automated memory leaks testing #995

even1024 opened this issue Jan 18, 2023 · 0 comments · Fixed by #1038
Assignees
@even1024
Milestone
Indigo-1.11.0-rc.1

Comments

@even1024
Copy link
Collaborator

Add memory leaks detection to Indogo's gtests

It can be implemented by wrapping of memory management C++ standard library functions as described below (both methods use the same technique).

https://www.gnu.org/software/libc/manual/html_node/Replacing-malloc.html
https://github.com/google/sanitizers/wiki/AddressSanitizer

There is an example of using the approach:

https://github.com/stephanfr/HeapWatcher
@even1024 even1024 added this to the Indigo-1.10.0-rc.1 milestone Jan 18, 2023
@even1024 even1024 self-assigned this Jan 18, 2023
@even1024 even1024 changed the title Memory leaks detection Automated memory leaks testing Jan 18, 2023
@even1024 even1024 mentioned this issue Jan 18, 2023
Open
@even1024 even1024 linked a pull request Feb 16, 2023 that will close this issue
#995 Automated memory leaks testing #1038
Merged
AlexanderSavelyev pushed a commit that referenced this issue Mar 7, 2023
@even1024
#995 Automated memory leaks testing (#1038)
6e92043 
* leaks

* leaks test

* test fix

* test fix

* libasan

* fix test

* fix test

* fix test

* Update Dockerfile

* docker fix

* docker fix

* test fix

* docker fix

* docker fix

* docker fix

* fix test

* leaks tests

* leaks tests

* leaks tests

* leaks tests

---------

Co-authored-by: Roman Porozhnetov <roman_porozhnetov@epam.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@even1024 even1024

Labels
None yet
Projects
None yet
Milestone
Indigo-1.11.0-rc.1
Development

Successfully merging a pull request may close this issue.

#995 Automated memory leaks testing epam/Indigo
2 participants
@even1024 @vanoprenko