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

Fix issues in code #79

Merged
merged 4 commits into from
Jul 29, 2024
Merged

Fix issues in code #79

merged 4 commits into from
Jul 29, 2024

Commits on Jul 28, 2024

  1. reassign variable passed to strreplace 

    realloc cannot guaranteed to use same location which is why the pointer needs to be reassigned
    @Jan200101
    Jan200101 committed Jul 28, 2024
    Configuration menu
    Copy the full SHA
    d700912 View commit details
    Browse the repository at this point in the history

  2. replace faulty strstr_i implementation 

    the original implementation tried driving p1 ahead resulting in an invalid read once we reach the null termination
fixed by checking p1 and incrementing it later
fixed bad function signature, we cannot return const char* as char*
    @Jan200101
    Jan200101 committed Jul 28, 2024
    Configuration menu
    Copy the full SHA
    c41b3fd View commit details
    Browse the repository at this point in the history

  3. free allocated memory after use

    @Jan200101
    Jan200101 committed Jul 28, 2024
    Configuration menu
    Copy the full SHA
    8e531a1 View commit details
    Browse the repository at this point in the history

  4. prevent buffer overrun when fetching non-steam apps

    @Jan200101
    Jan200101 committed Jul 28, 2024
    Configuration menu
    Copy the full SHA
    822bb00 View commit details
    Browse the repository at this point in the history