> The patch has been updated... The updates primarily consist of Alan's
> suggested changes below... (thank you) It applies against the 2.4.6
> kernel...
A quick glance through it:
dsp3780I_WriteDStore still touches user space with a spinlock held
(also doesnt check the get_user return)
The ioctl handlers do not check copy_from_user/to_user returns
IOCTL_MW_UNREGISTER_IPC will oops if fed bogus info (ipcnum should be
unsigned)
The return should be -ENOTTY not -ENOIOCTLCMD unless its internal code
that catches NOIOCTLCMD and changes it before user space sees it
mwave_Read should be -EINVAL not -ENOSYS (ENOSYS means the entire read syscall
in the OS isnt there)
In debug mode mwave_write accesses user space directly and may crash
(buf[0])
Trivial item - coding style uses foo(void) not foo() to indicate functions
taking no arguments
Still have globals like "dspio" "uartio" "ClaimResources" etc
whats wrong with tp3780_uart_io etc for globals ?
Otherwise it looks close to ready
Alan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Jul 15 2001 - 21:00:17 EST