From cbfa211988cd0baeeb5f095488be28a745b94389 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Thu, 1 Aug 2019 11:38:15 +0200 Subject: [PATCH] ProcFS: Align the buffer used for the CPUID brand string. I'm not sure if this actually matters, but it won't hurt anyone to use a 32-bit aligned buffer here. Found by PVS-Studio. --- Kernel/FileSystem/ProcFS.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Kernel/FileSystem/ProcFS.cpp b/Kernel/FileSystem/ProcFS.cpp index 93e717c49f..cac78f79af 100644 --- a/Kernel/FileSystem/ProcFS.cpp +++ b/Kernel/FileSystem/ProcFS.cpp @@ -478,7 +478,7 @@ ByteBuffer procfs$cpuinfo(InodeIdentifier) { // FIXME: Check first that this is supported by calling CPUID with eax=0x80000000 // and verifying that the returned eax>=0x80000004. - char buffer[48]; + alignas(u32) char buffer[48]; u32* bufptr = reinterpret_cast(buffer); auto copy_brand_string_part_to_buffer = [&](u32 i) { CPUID cpuid(0x80000002 + i);