fix: combine alert fixes 3235–3249#140
Conversation
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… converted to larger type Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
|
|
☑️ I checked the pre-commit hooks and there was nothing to fix at commit |
bniladridas
left a comment
There was a problem hiding this comment.
Added inline comments for overflow fix explanations
|
Review notes:
|
|
Merge was blocked because the inline review comments I added had unresolved threads. I resolved all three review threads in stb_image.h, stb_image_write.h and preprocess.c so the pull request is now ready to merge. Note All review threads must be resolved before a pull request can be merged on GitHub. |
Combines all 15 automated code scanning alert fixes for multiplication result converted to larger type. These changes span
include/stb_image.h,include/stb_image_write.h, andsrc/preprocess.cwith consistent application ofsize_tcasts to prevent integer overflow before conversion to larger types. Overflow checks are added where allocation sizes are stored back intointvariables such asout_size.Note
Merging this pull request will automatically close #125, #126, #127, #128, #129, #130, #131, #132, #133, #134, #135, #136, #137, #138, #139
Close #125
Close #126
Close #127
Close #128
Close #129
Close #130
Close #131
Close #132
Close #133
Close #134
Close #135
Close #136
Close #137
Close #138
Close #139
Discussion