Skip to content

r.in.pdal: fix new Coverity Scan issues - #7883

Merged
nilason merged 1 commit into
OSGeo:mainfrom
nilason:fix_r_in_pdal
Sep 4, 2026
Merged

nilason merged 1 commit into
OSGeo:mainfrom
nilason:fix_r_in_pdal

Conversation

@nilason

@nilason nilason commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Fix Coverity Scan issues introduced with #6465:

  • Uncaught exception of PointView::setField (CID 1685510)
  • Move reproject_option instead of passed-by-value for reproject->setOptions (CID1685509)

introduced with 1c68432

- Uncaught exception of PointView::setField (CID 1685510)
- Move reproject_option instead of passed-by-value for reproject->setOptions (CID1685509)
@nilason

nilason commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

A brief look at the diff gives the impression there are a lot of changes.
I only moved the try { up to line 141, and changed the line 182.

@github-actions github-actions Bot added raster Related to raster data processing C++ Related code is in C++ module labels Sep 4, 2026

@petrasovaa petrasovaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@nilason
nilason merged commit a768b68 into OSGeo:main Sep 4, 2026
26 checks passed
@nilason
nilason deleted the fix_r_in_pdal branch September 4, 2026 20:23
@github-actions github-actions Bot added this to the 8.6.0 milestone Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ Related code is in C++ module raster Related to raster data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants