Skip to content

[relro] Detect BIND_NOW via DT_FLAGS_1 (DF_1_NOW) and use bitmask for DF_BIND_NOW - #315

Merged
slimm609 merged 2 commits into
slimm609:mainfrom
zhaomaosu:fix-relro-check
Nov 19, 2025
Merged

[relro] Detect BIND_NOW via DT_FLAGS_1 (DF_1_NOW) and use bitmask for DF_BIND_NOW#315
slimm609 merged 2 commits into
slimm609:mainfrom
zhaomaosu:fix-relro-check

Conversation

@zhaomaosu

Copy link
Copy Markdown
Contributor

Add support for detecting RELRO/BIND_NOW through DT_FLAGS_1 (DF_1_NOW), in addition to DT_BIND_NOW and DT_FLAGS. Also switch DF_BIND_NOW check from equality (8) to a proper bitmask, and rename variables to Go-style camelCase for clarity.

…k for DF_BIND_NOW

Add support for detecting RELRO/BIND_NOW through DT_FLAGS_1 (DF_1_NOW),
in addition to DT_BIND_NOW and DT_FLAGS. Also switch DF_BIND_NOW check
from equality (8) to a proper bitmask, and rename variables to Go-style
camelCase for clarity.
@slimm609

Copy link
Copy Markdown
Owner

This has conflicts due to the recent merge of another bug around BIND_NOW, Can you please rebase this?

@zhaomaosu

Copy link
Copy Markdown
Contributor Author

This has conflicts due to the recent merge of another bug around BIND_NOW, Can you please rebase this?

Done. Rebased and resolved the conflicts.

@zhaomaosu

Copy link
Copy Markdown
Contributor Author

Kindly ping @slimm609 for review this PR.

@slimm609
slimm609 merged commit f33af03 into slimm609:main Nov 19, 2025
1 check passed
@zhaomaosu
zhaomaosu deleted the fix-relro-check branch November 25, 2025 01:25
@zhaomaosu

Copy link
Copy Markdown
Contributor Author

Hi @slimm609, do you plan to have an extra release to include this fix and #313 ? Since they are important to the basic functionality of the checksec tool. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants