Skip to content

fix: only keep RELRO_PADDING section when relro is enabled#1892

Merged
lapla-cogito merged 1 commit intowild-linker:mainfrom
TechnoPorg:push-vwvnvorqqxmp
May 5, 2026
Merged

fix: only keep RELRO_PADDING section when relro is enabled#1892
lapla-cogito merged 1 commit intowild-linker:mainfrom
TechnoPorg:push-vwvnvorqqxmp

Conversation

@TechnoPorg
Copy link
Copy Markdown
Contributor

@TechnoPorg TechnoPorg commented May 5, 2026

As mentioned in #1884 (comment), the check in apply_force_keep_sections to only keep the RELRO_PADDING section is overridden by the fact that the section is unconditionally kept a few lines above.

New ExpectProgramHeader/NoProgramHeader test directives have been added here to test this behaviour.

@TechnoPorg TechnoPorg force-pushed the push-vwvnvorqqxmp branch from ca49f83 to 072a6a0 Compare May 5, 2026 08:09
Comment thread wild/tests/integration_tests.rs Outdated
@TechnoPorg TechnoPorg force-pushed the push-vwvnvorqqxmp branch from 072a6a0 to 2132614 Compare May 5, 2026 10:55
Copy link
Copy Markdown
Member

@lapla-cogito lapla-cogito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@lapla-cogito lapla-cogito merged commit f559f56 into wild-linker:main May 5, 2026
46 of 47 checks passed
@TechnoPorg TechnoPorg deleted the push-vwvnvorqqxmp branch May 5, 2026 11:05
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