Skip to content

Fixes Crash in pcl::PLYReader::amendProperty#5331

Merged
mvieth merged 3 commits intoPointCloudLibrary:masterfrom
sashashura:patch-1
Jul 19, 2022
Merged

Fixes Crash in pcl::PLYReader::amendProperty#5331
mvieth merged 3 commits intoPointCloudLibrary:masterfrom
sashashura:patch-1

Conversation

@sashashura
Copy link
Copy Markdown
Contributor

Fixes Crash in pcl::PLYReader::amendProperty:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=42111 and
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=32184

In a pcl::PLYReader::amendProperty call cloud_->fields is empty and the finder iterator is invalid. It leads to invalid memory read in Release.

Fixes Crash in pcl::PLYReader::amendProperty:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=42111 and
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=32184

In a `pcl::PLYReader::amendProperty` call `cloud_->fields` is empty and the `finder` iterator is invalid. It leads to invalid memory read in Release.
Comment thread io/src/ply_io.cpp
@mvieth mvieth added module: io changelog: fix Meta-information for changelog generation labels Jul 19, 2022
Copy link
Copy Markdown
Member

@mvieth mvieth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@mvieth mvieth merged commit 7d52b10 into PointCloudLibrary:master Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: fix Meta-information for changelog generation module: io

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants