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

raster/r.statistics: Fix Resource Leak issue in o_sdev.c , o_skew.c , o_var.c , o_kurt.c #5077

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

ShubhamDesai
Copy link
Contributor

This pull request fixes issue identified by Coverity Scan (CID : 1207718, 1207719, 1207720)
Used G_free() to fix this issue.

@github-actions github-actions bot added raster Related to raster data processing C Related code is in C module labels Feb 9, 2025
@nilason nilason added this to the 8.5.0 milestone Feb 10, 2025
@nilason
Copy link
Contributor

nilason commented Feb 10, 2025

Please address 1207913 here to. With that one, all similar issues in the module r.statistics will be fixed.

(You can filter a directory with e.g. /raster/r.statistics/*).

@ShubhamDesai ShubhamDesai changed the title raster/r.statistics: Fix Resource Leak issue in o_sdev.c , o_skew.c , o_var.c raster/r.statistics: Fix Resource Leak issue in o_sdev.c , o_skew.c , o_var.c , o_kurt.c Feb 10, 2025
@ShubhamDesai
Copy link
Contributor Author

Please address 1207913 here to. With that one, all similar issues in the module r.statistics will be fixed.

(You can filter a directory with e.g. /raster/r.statistics/*).

Done

Copy link
Contributor

@nilason nilason left a comment

Choose a reason for hiding this comment

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

Thanks!

@nilason nilason enabled auto-merge (squash) February 10, 2025 20:29
@nilason nilason merged commit be43b74 into OSGeo:main Feb 10, 2025
27 of 28 checks passed
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
Development

Successfully merging this pull request may close these issues.

2 participants