Opened 9 years ago
Closed 6 years ago
#992 closed defect (fixed)
Memory Leaks
Reported by: | Georg Semmler | Owned by: | bbell |
---|---|---|---|
Priority: | minor | Milestone: | Future |
Component: | raslib | Version: | development |
Keywords: | Cc: | ||
Complexity: | Medium |
Description
Using gcc-asan while debugging my own c++-code (which uses rasdamans c++ interface) i discovered a few memory leaks in raslib, rasodmg and rnprotocol.
Change History (9)
comment:1 by , 9 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
comment:2 by , 9 years ago
Component: | undecided → raslib |
---|---|
Resolution: | invalid |
Status: | closed → reopened |
comment:4 by , 8 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
… due to lack of information on how to reproduce.
comment:5 by , 8 years ago
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
Steps to reproduce: Compile rasdaman with -fsanitize=address or -fsanitize=leak using a recent version of gcc or clang. See https://github.com/google/sanitizers/wiki/AddressSanitizer
Or simply lookup the source code location listed in comment 2. Most leaks are easy to spot. For example:
- http://rasdaman.org/browser/applications/rasgeo/wms-import/inittools/initpyramid.cc#L705
- http://rasdaman.org/browser/rasodmg/oqlquery.cc#L368
- http://rasdaman.org/browser/mddmgr/mddcoll.cc#L523
- http://rasdaman.org/browser/rnprotocol/rnpclientcomm.cc#L1096
- …
I've submitted a patch some time ago, that fixed some of those. It got rejected those days. I don't remember exactly why. If I remember correcty it was because I changed also some of the old unsafe str-functions to the safer strn ones.
http://rasdaman.org/patchmanager/rejected?patchop=Download+Selected-1687
comment:6 by , 8 years ago
Ok I thought you had a client program and identified the leaks with valgrind, so I was hoping you can attach the client code. Thanks for taking the time to provide more information.
comment:7 by , 8 years ago
Owner: | changed from | to
---|---|
Status: | reopened → assigned |
comment:8 by , 7 years ago
Owner: | changed from | to
---|
This is not closed.