Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

r.mask.status: fix null pointer dereference and false positive string overflow #4512

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

nilason
Copy link
Contributor

@nilason nilason commented Oct 14, 2024

Fixes two new issues reported by Coverity Scan:

  • Handles case of unsuccessful creation of json string
  • Silences false positive issue for string copy operation to buffer of size GMAPSET_MAX from G_mapset()

… overflow

Fixes two new issues reported by Coverity Scan:

- Handles case of unsuccessful creation of json string
- Silences false positive issue for string copy operation to buffer of size GMAPSET_MAX from G_mapset().
@nilason nilason added this to the 8.5.0 milestone Oct 14, 2024
@github-actions github-actions bot added raster Related to raster data processing C Related code is in C libraries module labels Oct 14, 2024
Copy link
Member

@wenzeslaus wenzeslaus left a comment

Choose a reason for hiding this comment

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

Thanks for fixing it! I'm happy the detection works.

@wenzeslaus wenzeslaus merged commit 61b380f into OSGeo:main Oct 14, 2024
27 checks passed
@nilason nilason deleted the fix_r_mask_status branch February 4, 2025 14:55
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 libraries module raster Related to raster data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants