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

Classifier crash in noise filter #21

Open
dmiklic opened this issue Jul 5, 2017 · 2 comments
Open

Classifier crash in noise filter #21

dmiklic opened this issue Jul 5, 2017 · 2 comments

Comments

@dmiklic
Copy link
Contributor

dmiklic commented Jul 5, 2017

The module seems to be crashing on the robot. Also, apparently the module leaks memory.
@robotiks , @fpetric and @stribor14 please provide relevant details.

@dmiklic dmiklic changed the title Classifier cr@ash in noise filter Classifier crash in noise filter Jul 5, 2017
@fpetric
Copy link
Contributor

fpetric commented Jul 6, 2017

The problem seems to originate from the insertWindow function in noiseFilter.cpp, but arises only on the robot (running the module as remote does not produce this error on my setup). It may be specific to cross-compilation toolchain.

The error in question comes from glibc, and it looks something like this:
*** glibc detected *** ./libSCModule.so: double free or corruption (out): 0xXXXXXXXX ***
which would seem to indicate heap error of some sort. I suspected that the error originates from one of the two delete calls at the top of the insertWindow function in noiseFilter.cpp, so I tried commenting them and got a different error from glibc, memory corruption or something like that.

@fpetric
Copy link
Contributor

fpetric commented Jul 6, 2017

Regarding the memory leak, maybe we should look to re-implement the noise filter using boost buffers instead of plain arrays? @stribor14 any thoughts on this?

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