--Properly initialize std::vector member variable to be correct size - Githubissues

facebookresearch / habitat-sim

A flexible, high-performance 3D simulator for Embodied AI research.

https://aihabitat.org/

MIT License

2.45k stars 404 forks source link

--Properly initialize std::vector member variable to be correct size #2388

Closed jturner65 closed 2 months ago

jturner65 commented 2 months ago

Motivation and Context

This bugfix fixes the incorrect initialization of the SceneNode semanticIDs_ vector which was causing a heap corruption.

How Has This Been Tested

Local c++ and python tests pass

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.