Skip to content
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

将来的にメモリが保護されない恐れがある #34

Open
ndifix opened this issue Feb 9, 2022 · 2 comments
Open

将来的にメモリが保護されない恐れがある #34

ndifix opened this issue Feb 9, 2022 · 2 comments

Comments

@ndifix
Copy link

ndifix commented Feb 9, 2022

  • 問題点
    kUEFIPageSize が kBytesPerFrame よりも細分化されていた場合、
    プロセスに対して割り当てるべきでないメモリ領域が割り当てられる可能性がある。
    ただし現状、両者の定数は同じ値が指定されているのでここで指摘する問題は起こらない。

より具体的な状況として、1つの frame の中に 使用可能な page と不可能な page が混在している場合に frame 全体が使用可能なメモリ領域として扱われる可能性がある。

@uchan-nos
Copy link
Owner

ご報告ありがとうございます。
kUEFIPageSize が 4096 より小さくなることは考え難いですが、kBytesPerFrame を 4096 より大きくしてしまうことは考えられますね。2MB ページや 1GB ページを使いたくなるなど。
static_assert か何かで、kBytesPerFrame <= kUEFIPageSize であることをチェックするようにしても良いかもしれません。

@ndifix
Copy link
Author

ndifix commented Feb 13, 2022

ページサイズを大きくすることを許さないのであればstatic_assert で対応するのは簡潔でいいですね

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

No branches or pull requests

2 participants