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

A memory leak in VFRRT #839

Closed
bromiao123 opened this issue Oct 7, 2021 · 2 comments
Closed

A memory leak in VFRRT #839

bromiao123 opened this issue Oct 7, 2021 · 2 comments

Comments

@bromiao123
Copy link

When I’m testing ompl, here is a memory leak occured. After positioning,we found that the error is caused by the following code in VFRRT.cpp:

图片1

In line 169.A motion object was requested with it’s constructor,and also new a state in that motion.but in line 170, the state in motion was assigned. The space applied in the constructor becomes a wild space.That’s why memory leaks.
We suggest to use the default constructor in line 169.like that:
图片2

I did my experiment on Ubuntu 16.04

@mamoll mamoll closed this as completed in abb4fad Oct 7, 2021
@mamoll
Copy link
Member

mamoll commented Oct 7, 2021

Thanks for tracking down this leak!

@carnil
Copy link

carnil commented May 9, 2022

This issue appears to have CVE-2021-42218 assigned.

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

3 participants