-
Notifications
You must be signed in to change notification settings - Fork 21
fix: add security hardening flags to build configuration #273
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.
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
Conversation
1. Added DEB_BUILD_MAINT_OPTIONS with hardening=+all for comprehensive security 2. Included -Wall flag for both C and C++ compilers to enable all warnings 3. Added multiple linker security flags (-Wl options) for RELRO, immediate binding, and noexecstack protection 4. Maintained existing -Ofast optimization while adding security enhancements These changes improve the security of the built package by enabling modern compiler and linker protections against common vulnerabilities like buffer overflows and code injection attacks. fix: 在构建配置中添加安全加固标志 1. 添加 DEB_BUILD_MAINT_OPTIONS 并设置 hardening=+all 以实现全面的安全 保护 2. 为 C 和 C++ 编译器添加 -Wall 标志以启用所有警告 3. 添加多个链接器安全标志 (-Wl 选项) 用于 RELRO、立即绑定和 noexecstack 保护 4. 在保持现有 -Ofast 优化的同时增加安全增强 这些更改通过启用现代编译器和链接器保护来防止常见漏洞(如缓冲区溢出和代码 注入攻击),从而提高了构建包的安全性。
Reviewer's GuideBuild configuration updated to enable comprehensive security hardening by setting DEB_BUILD_MAINT_OPTIONS, augmenting compiler warning flags, and tightening linker protections while preserving existing optimizations. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @18202781743 - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
deepin pr auto review关键摘要:
是否建议立即修改:
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, mhduiy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
security
warnings
immediate binding, and noexecstack protection
enhancements
These changes improve the security of the built package by enabling
modern compiler and linker protections against common vulnerabilities
like buffer overflows and code injection attacks.
fix: 在构建配置中添加安全加固标志
保护
保护
这些更改通过启用现代编译器和链接器保护来防止常见漏洞(如缓冲区溢出和代码
注入攻击),从而提高了构建包的安全性。
Summary by Sourcery
Build: