Bug fixes (Memory leaks, default permissions...)

GhiXu / ACMP

Planar Prior Assisted PatchMatch Multi-View Stereo

MIT License

188 stars 30 forks source link

Closed rjanvier closed 1 year ago

rjanvier commented 3 years ago

fix default permissions (777 is a bad idea as it's way too much permissive)
correct format for size_t
fix an out of array bounds access
fix leaks as cv::Mat does not take ownership of data pointer and thus cannot de allocate the underlying memory.

same fixes hold for ACMM and ACMH. I'll make a dedicated PR for the two other repos as well if you are ok with that.

GhiXu commented 1 year ago

Thank you very much for your help!