Skip to content

Conversation

ShubhamDesai
Copy link
Contributor

This pull request fixes issue identified by Coverity Scan (CID : 1208276)

@github-actions github-actions bot added C Related code is in C module imagery labels Oct 7, 2025
ShubhamDesai and others added 4 commits October 7, 2025 13:19
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nilason nilason added this to the 8.5.0 milestone Oct 7, 2025
@github-project-automation github-project-automation bot moved this to In Progress in GRASS Security Oct 7, 2025
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.

Oh, there is a similar issue CID1208244, please address that here too.

@ShubhamDesai
Copy link
Contributor Author

Oh, there is a similar issue CID1208244, please address that here too.

this issue is shown under /vector/v.rectify/target.c

or are you talking about the strcat at line 292

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

If you filter with /imagery/i.ortho.photo/i.ortho.rectify/* you should see which one I mean.

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

(Coverity scan is confused by similarly named files located in different places).

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

/imagery/i.ortho.photo/i.ortho.rectify/target.c

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@ShubhamDesai
Copy link
Contributor Author

Oh, there is a similar issue CID1208244, please address that here too.

Done

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 imagery module

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants