patch to make mdev take notice of file perms in mdev.conf

Stuart Hughes stuarth at freescale.com
Fri Sep 26 08:05:16 UTC 2008


On Thu, 2008-09-25 at 19:27 +0200, Bernhard Reutner-Fischer wrote:
> On Thu, Sep 25, 2008 at 04:56:15PM +0100, Stuart Hughes wrote:
> >Here's a patch against 1.11.2 that changes mdev so that it will set the
> >file permissions on a device node that are given in mdev.conf.
> >
> >Regards, Stuart
> 
> >20080925 Stuart Hughes <stuarth at freescale dot com>
> >
> >Make mdev take notice of file mode in mdev.conf
> >
> >diff --exclude CVS -uNr busybox-1.11.2/util-linux/mdev.c busybox-1.11.2.modified/util-linux/mdev.c
> >--- busybox-1.11.2/util-linux/mdev.c	2008-06-25 13:51:32.000000000 +0100
> >+++ busybox-1.11.2.modified/util-linux/mdev.c	2008-09-25 15:36:33.000000000 +0100
> >@@ -254,6 +254,7 @@
> > 
> > 		if (ENABLE_FEATURE_MDEV_CONF) {
> > 			chown(device_name, uid, gid);
> >+			chmod(device_name, mode);
> 
> are you sure?
> A few lines above, mknod is called with the proper mode already?

You're right.  What's happening on my system is that /dev/console,null
are being made in a startup script before mdev and the umask is
clobbering them.  When mdev runs, it doesn't make these nodes as they're
already there.

I'll fix my startup scripts.  Apologies for the noise.

Regards, Stuart










More information about the busybox mailing list