XyrisOS/xyris

Wrong macro used for loop condition in paging.cpp:initDirectory()

Closed this issue · 1 comments

vannaka commented

xyris/Kernel/Memory/paging.cpp

Line 134 in adf3ae0

for (size_t j = 0; j < ARCH_PAGE_DIR_ENTRIES; j++) {

I believe the upper bound for the iterator of this for loop is meant to be ARCH_PAGE_TABLE_ENTRIES. The current code works because the two macros happen to have the same value.

Kfeavel commented

You would be correct! Thank you for opening an issue, I'll merge in a fix soon.