diff options
author | Keegan Saunders <keegan@undefinedbehaviour.org> | 2022-11-26 13:17:32 -0500 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2022-11-29 11:04:21 +0100 |
commit | 89b23c473a57ddc37c316cca9b21d7a636e86d81 (patch) | |
tree | 73635c650a4b19949ca24742bf1c170864af9aba /Userland/Libraries/LibELF | |
parent | 675e5bfdcebb41eb172027824ad46ab056c2730e (diff) | |
download | serenity-89b23c473a57ddc37c316cca9b21d7a636e86d81.zip |
LibC: Use uintptr_t for __stack_chk_guard
We used size_t, which is a type that is guarenteed to be large
enough to hold an array index, but uintptr_t is designed to be used
to hold pointer values, which is the case of stack guards.
Diffstat (limited to 'Userland/Libraries/LibELF')
-rw-r--r-- | Userland/Libraries/LibELF/DynamicLinker.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Userland/Libraries/LibELF/DynamicLinker.cpp b/Userland/Libraries/LibELF/DynamicLinker.cpp index 1658d99c36..ee4ee79eac 100644 --- a/Userland/Libraries/LibELF/DynamicLinker.cpp +++ b/Userland/Libraries/LibELF/DynamicLinker.cpp @@ -258,7 +258,7 @@ static void initialize_libc(DynamicObject& libc) // This is not done in __libc_init, as we definitely have to return from that, and it might affect Loader as well. res = libc.lookup_symbol("__stack_chk_guard"sv); VERIFY(res.has_value()); - arc4random_buf(res.value().address.as_ptr(), sizeof(size_t)); + arc4random_buf(res.value().address.as_ptr(), sizeof(uintptr_t)); res = libc.lookup_symbol("__environ_is_malloced"sv); VERIFY(res.has_value()); |