On Sun, 28 Nov 2021 14:14:26 +0100 Markus Theil <markus.th...@tu-ilmenau.de> wrote:
> From: Markus Theil <markus.th...@secunet.com> > > Fix kni's ioctl signature to correctly match the kernel's > structs. This shaves off the (void*) casts and uses struct file* > instead of struct inode*. With the correct signature, control flow > integrity checkers are no longer confused at this point. > > Signed-off-by: Markus Theil <markus.th...@secunet.com> > Tested-by: Michael Pfeiffer <michael.pfeif...@tu-ilmenau.de> > --- > v2: adapt to suggestions from Ferruh Yigit > > kernel/linux/kni/kni_misc.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > -static int > -kni_compat_ioctl(struct inode *inode, uint32_t ioctl_num, > +static long > +kni_compat_ioctl(struct file *file, uint32_t ioctl_num, > unsigned long ioctl_param) > { Shouldn't be uint32_t should be unsigned int. In fs.h. long (*compat_ioctl) (struct file *, unsigned int, unsigned long);