Re: World writeable MUX

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Felipe Balbi <balbi@xxxxxx> [110204 04:42]:
> Hi Tony,
> 
> I saw a few patches floating around regarding world writeable sysfs
> files and decided to grep around arch/arm/*omap* to try and see if we
> were safe. Turns out anyone can change MUX entries via the debugfs
> interface:
> 
> static void __init omap_mux_dbg_create_entry(
> 				struct omap_mux_partition *partition,
> 				struct dentry *mux_dbg_dir)
> {
> 	struct omap_mux_entry *e;
> 
> 	list_for_each_entry(e, &partition->muxmodes, node) {
> 		struct omap_mux *m = &e->mux;
> 
> 		(void)debugfs_create_file(m->muxnames[0], S_IWUGO, mux_dbg_dir,
> 					  m, &omap_mux_dbg_signal_fops);
> 	}
> }
> 
> is that really expected ? I guess not, attached there's a small patch to
> fix that one.

Heh, no it should not be like that, thanks will queue as a fix.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux