PointCloudLibrary/pcl

Fix Memory Leaks

Open

#3612 aperta il 30 gen 2020

Vedi su GitHub
 (8 commenti) (1 reazione) (0 assegnatari)C++ (4506 fork)batch import
good first issuehelp wantedmodule: octreestatus: stale

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

Your Environment

  • PCL Version: HEAD, CI

Context

Most of the leaks are in application/test code, which will not impact the core library. But they should be removed nonetheless.

Some of the leaks have been fixed, but there are still some left.

List of files with leaks

ml/src/svm.cpp:2354  error memleakOnRealloc: Common realloc mistake: 'label' nulled but not freed upon failure
ml/src/svm.cpp:2355  error memleakOnRealloc: Common realloc mistake: 'count' nulled but not freed upon failure
ml/src/svm.cpp:3122  error memleakOnRealloc: Common realloc mistake: 'line' nulled but not freed upon failure
ml/src/svm.cpp:3523  error memleakOnRealloc: Common realloc mistake: 'label' nulled but not freed upon failure
ml/src/svm.cpp:3524  error memleakOnRealloc: Common realloc mistake: 'count' nulled but not freed upon failure
ml/src/svm.cpp:3181  error memleak: Memory leak: model.scaling
people/apps/main_ground_based_people_detection.cpp:255  error memleak: Memory leak: interface
surface/src/3rdparty/opennurbs/opennurbs_beam.cpp:868  warning nullPointerRedundantCheck: Either the condition '0!=polycurve' is redundant or there is possible null pointer dereference: polycurve.
surface/src/3rdparty/opennurbs/opennurbs_beam.cpp:2517  error nullPointer: Possible null pointer dereference: k
surface/src/3rdparty/opennurbs/opennurbs_beam.cpp:2962  error nullPointer: Possible null pointer dereference: null_brep_pointer
surface/src/3rdparty/opennurbs/opennurbs_beam.cpp:2936  warning nullPointerRedundantCheck: Either the condition 'newbrep' is redundant or there is possible null pointer dereference: newbrep.
surface/src/3rdparty/opennurbs/opennurbs_brep_tools.cpp:1269  warning nullPointerRedundantCheck: Either the condition '!brep' is redundant or there is possible null pointer dereference: brep.
surface/src/3rdparty/opennurbs/opennurbs_brep_tools.cpp:1860  warning nullPointerRedundantCheck: Either the condition '!Srf' is redundant or there is possible null pointer dereference: Srf.
surface/src/3rdparty/opennurbs/opennurbs_polycurve.cpp:2803  warning nullPointerRedundantCheck: Either the condition 'pLeftSide' is redundant or there is possible null pointer dereference: pLeftSide.
surface/src/3rdparty/opennurbs/opennurbs_polycurve.cpp:2808  warning nullPointerRedundantCheck: Either the condition 'pRightSide' is redundant or there is possible null pointer dereference: pRightSide.
surface/src/3rdparty/poisson4/marching_cubes_poisson.cpp:752  error uninitvar: Uninitialized variable: v
surface/src/3rdparty/poisson4/marching_cubes_poisson.cpp:867  error uninitvar: Uninitialized variable: v
surface/src/3rdparty/poisson4/marching_cubes_poisson.cpp:883  error uninitvar: Uninitialized variable: v
test/2d/test_2d.cpp:124  error memleak: Memory leak: k
test/2d/test_2d.cpp:124  error memleak: Memory leak: conv
test/2d/test_2d.cpp:157  error memleak: Memory leak: k
test/2d/test_2d.cpp:157  error memleak: Memory leak: conv
test/2d/test_2d.cpp:296  error memleak: Memory leak: morph
test/2d/test_2d.cpp:329  error memleak: Memory leak: morph
test/2d/test_2d.cpp:362  error memleak: Memory leak: morph
test/2d/test_2d.cpp:395  error memleak: Memory leak: morph
test/io/test_octree_compression.cpp:111  error memleak: Memory leak: pointcloud_encoder
test/io/test_octree_compression.cpp:111  error memleak: Memory leak: pointcloud_decoder
test/io/test_octree_compression.cpp:156  error memleak: Memory leak: pointcloud_encoder
test/io/test_octree_compression.cpp:156  error memleak: Memory leak: pointcloud_decoder
test/search/test_auto_search.cpp:187  error memleak: Memory leak: octree
test/search/test_auto_search.cpp:311  error memleak: Memory leak: kdtree
test/search/test_auto_search.cpp:309  error memleak: Memory leak: kdtree
test/search/test_auto_search.cpp:325  error memleak: Memory leak: search
test/search/test_auto_search.cpp:344  error memleak: Memory leak: kdtree
test/search/test_flann_search.cpp:128  error memleak: Memory leak: FlannSearch
test/search/test_flann_search.cpp:127  error memleak: Memory leak: FlannSearch
test/search/test_flann_search.cpp:174  error memleak: Memory leak: FlannSearch
test/search/test_flann_search.cpp:207  error memleak: Memory leak: FlannSearch
test/search/test_flann_search.cpp:252  error memleak: Memory leak: flann_search
test/search/test_flann_search.cpp:378  error memleak: Memory leak: FlannSearch
test/search/test_kdtree.cpp:123  error memleak: Memory leak: kdtree
test/search/test_kdtree.cpp:122  error memleak: Memory leak: kdtree
test/search/test_kdtree.cpp:166  error memleak: Memory leak: kdtree
test/search/test_kdtree.cpp:196  error memleak: Memory leak: kdtree
test/search/test_kdtree.cpp:208  error memleak: Memory leak: kdtree
test/search/test_octree.cpp:167  error memleak: Memory leak: octree
test/search/test_organized_index.cpp:173  error memleak: Memory leak: organizedNeighborSearch
test/search/test_organized_index.cpp:202  error memleak: Memory leak: organizedNeighborSearch
test/search/test_organized_index.cpp:375  error memleak: Memory leak: organizedNeighborSearch
test/search/test_organized_index.cpp:461  error memleak: Memory leak: organizedNeighborSearch
test/search/test_organized_index.cpp:600  error memleak: Memory leak: organizedNeighborSearch
test/test_recognition_ransac_based_ORROctree.cpp:117  error memleak: Memory leak: new_model
tools/mesh_sampling.cpp:92  error nullPointer: Possible null pointer dereference: ptIds
tools/mesh_sampling.cpp:93  error nullPointer: Possible null pointer dereference: ptIds
tools/mesh_sampling.cpp:94  error nullPointer: Possible null pointer dereference: ptIds
tools/mesh_sampling.cpp:115  error nullPointer: Possible null pointer dereference: ptIds
tools/mesh_sampling.cpp:116  error nullPointer: Possible null pointer dereference: ptIds
tools/mesh_sampling.cpp:117  error nullPointer: Possible null pointer dereference: ptIds
tools/oni_viewer_simple.cpp:210  error memleak: Memory leak: grabber
visualization/src/point_cloud_handlers.cpp:741  error memleak: Memory leak: pts

The format allows most editors to open the file using Ctrl+click (no, not vim :stuck_out_tongue: )

Guida contributor