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

Fix invalid usage of safeDelete which can cause an error with valgrin… #544

Closed
wants to merge 1 commit into from
Closed

Fix invalid usage of safeDelete which can cause an error with valgrin… #544

wants to merge 1 commit into from

Conversation

Touyote
Copy link

@Touyote Touyote commented Jun 1, 2017 •
edited

Fixed an invalid usage of safeDelete which can cause a valgrind & libasan error.

Signed-off-by: Romain GASQ

This is a

  • Breaking change
  • New feature
  • [ x] Bugfix

I have

Fix invalid usage of safeDelete which can cause an error with valgrin…
6b07a67 
…d & libasan

Signed-off-by: Romain GASQ <r.gasq@ssi.samsung.com>
@Touyote
Copy link
Author

Touyote commented Jun 23, 2017

For a dynamic generation/managing of module logger during the runtime, it could be very cool to integrate that patch for avoiding the unspecified crashed which can be cause by the library.

@abumq
Copy link
Owner

abumq commented Aug 2, 2017

Doing manually, see 17a9e3b

@abumq abumq closed this Aug 2, 2017
@Touyote
Copy link
Author

Touyote commented Aug 16, 2017

Thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Touyote @abumq