8000 Fix: memory leaks, potential ub, buffer overflows, cppcheck and compiler warnings by dnzbk · Pull Request #552 · nzbgetcom/nzbget · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix: memory leaks, potential ub, buffer overflows, cppcheck and compiler warnings #552

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
Merged
merged 1 commit into from
Apr 29, 2025
Merged

Fix: memory leaks, potential ub, buffer overflows, cppcheck and compiler warnings #552

merged 1 commit into from
Apr 29, 2025

Conversation

dnzbk
Copy link
Collaborator
@dnzbk dnzbk commented Apr 25, 2025 •
edited
Loading

Description

  • Resolved some found memory leaks
  • Fixed potential buffer overflows and undefined behavior issues
  • Resolved many Cppcheck and compiler warnings
  • Added ENABLE_SANITIZERS CMake option to build with Address, Memory and UndefinedBehavior sanitizers for easier debugging of memory issues
  • updated the POSIX Documentation:
    • Added info about the new ENABLE_SANITIZERS option

Testing

  • macOS Ventura
  • Windows 11
  • Linux Debian 12

@dnzbk dnzbk requested review from phnzb and luckedea April 25, 2025 06:24
@dnzbk dnzbk mentioned this pull request Apr 25, 2025
Closed
1 task
@dnzbk dnzbk force-pushed the fix/memory-leaks branch from 817b11b to 0c9acae Compare April 28, 2025 07:22
@dnzbk dnzbk merged commit 3baf9bd into develop Apr 29, 2025
9 checks passed
@dnzbk dnzbk deleted the fix/memory-leaks branch April 29, 2025 04:32
@dnzbk dnzbk mentioned this pull request May 12, 2025
Merged
@BrewTestBot BrewTestBot mentioned this pull request May 12, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luckedea luckedea luckedea approved these changes

@phnzb phnzb Awaiting requested review from phnzb

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dnzbk @luckedea
0