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.

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

Memory leaks in data_resumption_test unit tests #2272

Closed
shoamano83 opened this issue Jun 18, 2018 · 2 comments
Closed

Memory leaks in data_resumption_test unit tests #2272

shoamano83 opened this issue Jun 18, 2018 · 2 comments

Comments

@shoamano83
Copy link
Contributor

Bug Report

Memory leaks are observed when data_resumption_test is executed under valgrind.

Reproduction Steps
  1. Check out commit 1527db9 from develop branch
  2. Give a build with cmake ../sdl_core -DBUILD_TESTS=ON and make
  3. under src/components/application_manager/test directory, run valgrind --trace-children=yes --leak-check=full ./data_resumption_test > data_resumption_test_valgrind.txt 2>&1
Expected Behavior

Memory leaks should not be observed.

Observed Behavior

Valgrind reports several memory leaks. Please refer to attached log files.

OS & Version Information
  • OS/Version: Ubuntu 16.04 (amd64)
  • SDL Core Version: Commit 1527db9 from develop branch
  • Testing Against: N/A (unit testing)
Test Case, Sample Code, and / or Example App

Valgrind log is attached: data_resumption_test_valgrind.txt

@shoamano83
Copy link
Contributor Author

I created a fix for this issue to find out that the memory leaks were caused by inappropriate implementation of the unit test code. This is not an issue of SDL Core itself, so the priority may be low.

@iCollin
Copy link
Collaborator

iCollin commented Jun 6, 2020

Closed via #2273

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

No branches or pull requests

2 participants