Re: [PATCH v5 8/8] module: replace the existing LSM hook in init_module
From: kbuild test robot
Date: Tue Jul 03 2018 - 05:36:43 EST
Hi Mimi,
I love your patch! Perhaps something to improve:
[auto build test WARNING on integrity/next-integrity]
[also build test WARNING on v4.18-rc3 next-20180702]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Mimi-Zohar/kexec-firmware-support-system-wide-policy-requiring-signatures/20180703-011114
base: https://git.kernel.org/pub/scm/linux/kernel/git/zohar/linux-integrity.git next-integrity
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> security/loadpin/loadpin.c:178:40: sparse: mixing different enum types
security/loadpin/loadpin.c:178:40: int enum kernel_load_data_id versus
security/loadpin/loadpin.c:178:40: int enum kernel_read_file_id
vim +178 security/loadpin/loadpin.c
175
176 static int loadpin_load_data(enum kernel_load_data_id id)
177 {
> 178 return loadpin_read_file(NULL, id);
179 }
180
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation