Skip to content

Fixed bug stemming from treating an exclusive range as an inclusive r…#362

Merged
phil-opp merged 1 commit intorust-osdev:mainfrom
JarlEvanson:main
Apr 9, 2023
Merged

Fixed bug stemming from treating an exclusive range as an inclusive r…#362
phil-opp merged 1 commit intorust-osdev:mainfrom
JarlEvanson:main

Conversation

@JarlEvanson
Copy link
Contributor

This should fix a rare bug regarding treating an exclusive range as an inclusive range when the zero_end is page aligned.

Copy link
Member

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

@phil-opp phil-opp merged commit bb92e9e into rust-osdev:main Apr 9, 2023
@phil-opp phil-opp mentioned this pull request Jul 5, 2023
phil-opp added a commit that referenced this pull request Feb 16, 2024
The `zero_end` bound is exclusive, but we treat the `end_page` as inclusive. So when `zero_end` is page-aligned, we allocate one additional bss page. If this page was already mapped to some other segment, we remap it to a page with random content.

This is the same bug as #362.
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