Thread: Do I need to delete dynamic memory before leaving a function

Ok, I'm pretty sure that I'll get my head bitten of for this but if you are going to, please at least point me to a webpage that explains it.

I am using Point Cloud Library to manipulate point clouds (just to give the context). From my main I call a function and withing that function I use

Code:

pcl::PointCloud<pcl::PointXYZRGB>::Ptr cloudIn(new pcl::PointCloud<pcl::PointXYZRGB>());

It is essentially a class that acts as a pointer and the constructor needs the regular class as its argument. My question is this: do I need to delete the new pcl::PointCloud<pcl::PointXYZRGB> object before I leave the function or will it delete this by itself? I can run it just fine without doing so but I'm not sure if this causes memory leaks (I've encountered a lot of articles about this on the internet)

If you're sure that the library won't require the abject later on, deleting it is fine.
You could also use smart pointers...which will automatically destroy it when not needed.

I just found out experimentally that if I try to delete it just before I leave the function, I get a segmentation fault. This fault happens after the last line of the function and before the first line the main after calling the function. It seems that I cannot delete it without problems but that would seem to contradict what i have read about dynamically allocated memory and deleting.

@manasij7479 What is a smart pointer? How would you implement one?

Originally Posted by njitram2000

I just found out experimentally that if I try to delete it just before I leave the function, I get a segmentation fault. This fault happens after the last line of the function and before the first line the main after calling the function. It seems that I cannot delete it without problems but that would seem to contradict what i have read about dynamically allocated memory and deleting.

That just means that it is needed afterwards.

@manasij7479 What is a smart pointer? How would you implement one?

I've never implemented one...but a popular way is using reference counting.(Google the term)
But unless you're sure that you can do better, stick with the ones provided by the C++ standard.
Smart pointer - Wikipedia, the free encyclopedia

@anon & Elysia - Thank you, that explains everything. I looked through the PCL manual but not the Boost one (seemed a bit too daunting )
I did that now and I'll remember that tip for in the future.