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.

Sign up for GitHub

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

Jump to bottom

v.generalize: Initialize all of structure contents before using it #4281

Merged
merged 2 commits into from
Sep 10, 2024
Merged

v.generalize: Initialize all of structure contents before using it #4281

merged 2 commits into from
Sep 10, 2024

Commits on Sep 5, 2024

  1. v.generalize: Initialize all of structure contents before using it 

    Currently, in `head` which is a `POINT_LIST` structure, we are
only initializing the next pointer to NULL, but data present
in the point substructure is not initialized which implies that
it would be filled with random data. To avoid such scenario,
initialize point coordinates to zero.

This was found using cppcheck tool.

Signed-off-by: Mohan Yelugoti <ymdatta.work@gmail.com>
    @ymdatta
    ymdatta committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    644b403 View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2024

  1. Use correct literal for double type 

    Co-authored-by: Nicklas Larsson <n_larsson@yahoo.com>
    @ymdatta @nilason
    ymdatta and nilason authored Sep 10, 2024
    Configuration menu
    Copy the full SHA
    84faa5d View commit details
    Browse the repository at this point in the history