EDAC/{sb,i7core}_edac: Do not use a plain integer for a NULL pointer
authorAbhinav Singh <singhabhinav9051571833@gmail.com>
Tue, 28 Nov 2023 14:17:03 +0000 (19:47 +0530)
committerBorislav Petkov (AMD) <bp@alien8.de>
Tue, 28 Nov 2023 14:43:43 +0000 (15:43 +0100)
commita2f99fbae4513ce4871e71ba88cd9d75fe5019e6
treeae8590d7bcc9871cda6e7b7768515939524451f4
parent9e08ac1b5e3bb7837aa23b67a97f08b80384e9d2
EDAC/{sb,i7core}_edac: Do not use a plain integer for a NULL pointer

Sparse warns about the use of the integer constant 0 as a NULL pointer
with the -Wnon-pointer-null switch.

Even though the C standard requires that 0 == NULL and type conversion
rules turn an integer constant 0 into a NULL pointer when cast to a void
* type, Linus notes that this is a very poor situation from a type
safety angle and a pointer should be initialized with a pointer type
- not an integer constant.

See https://www.spinics.net/lists/linux-sparse/msg10066.html for more
info.

  [ bp: Rewrite commit message, drop useless comments in the code. ]

Signed-off-by: Abhinav Singh <singhabhinav9051571833@gmail.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/r/20231128141703.614605-1-singhabhinav9051571833@gmail.com
drivers/edac/i7core_edac.c
drivers/edac/sb_edac.c