Messages in this thread |  | | Date | Tue, 14 Nov 2000 11:58:52 +0000 (GMT) | From | Chris Evans <> | Subject | Re: Modprobe local root exploit |
| |
On Tue, 14 Nov 2000, Jakub Jelinek wrote:
> > Rather than add sanity checking to modprobe, it would be a lot easier > > and safer from a security audit point of view to have the kernel call > > /sbin/kmodprobe instead of /sbin/modprobe. Then kmodprobe can sanitise > > all the data and exec the real modprobe. That way the only thing that > > needs auditing is a string munging/sanitising program. > > Well, no matter what kernel needs auditing as well, the fact that dev_load > will without any check load any module the user wants is already problematic > and no munging helps with it at all, especially loading old ISA drivers > might not be a good idea.
FWIW: A quick look at the kernel source, and dev_load() seems to be the only place that does this. Other places apply prefixes to user supplied names.
Cheers Chris
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
|  |