x86/pci: Fix the function type for check_reserved_t
authorSami Tolvanen <samitolvanen@google.com>
Mon, 30 Nov 2020 19:39:00 +0000 (11:39 -0800)
committerBorislav Petkov <bp@suse.de>
Tue, 1 Dec 2020 13:22:52 +0000 (14:22 +0100)
commit83321c335dccba262a57378361d63da96b8166d6
tree413fe01da217e91ab5b7febe3107292dd6b84bb9
parentaeaaf005da1de075929e56562dced4a58238efc4
x86/pci: Fix the function type for check_reserved_t

e820__mapped_all() is passed as a callback to is_mmconf_reserved(),
which expects a function of type:

  typedef bool (*check_reserved_t)(u64 start, u64 end, unsigned type);

However, e820__mapped_all() accepts enum e820_type as the last argument
and this type mismatch trips indirect call checking with Clang's
Control-Flow Integrity (CFI).

As is_mmconf_reserved() only passes enum e820_type values for the
type argument, change the typedef and the unused type argument in
is_acpi_reserved() to enum e820_type to fix the type mismatch.

Reported-by: Sedat Dilek <sedat.dilek@gmail.com>
Suggested-by: Borislav Petkov <bp@alien8.de>
Signed-off-by: Sami Tolvanen <samitolvanen@google.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20201130193900.456726-1-samitolvanen@google.com
arch/x86/pci/mmconfig-shared.c