[PATCH 3/3] Remove umask temporarily so device permissions are correct (#383531, wmealing).

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

 



---
 loader/init.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/loader/init.c b/loader/init.c
index cc4ca68..1c275d8 100644
--- a/loader/init.c
+++ b/loader/init.c
@@ -348,6 +348,11 @@ static int termcmp(struct termios *a, struct termios *b) {
 static void createDevices(void) {
     int i;
 
+    /*	unset the umask so devices are created with correct perms
+	and not complemented by the previous umask call */
+
+    mode_t previous_umask = umask(0); 
+
     for (i = 0; devnodes[i].devname != NULL; i++) {
         char devname[64];
         int type = -1;
@@ -373,6 +378,9 @@ static void createDevices(void) {
                   makedev(devnodes[i].major, devnodes[i].minor)) < 0)
             fprintf(stderr, "Unable to create device %s: %m\n", devname);
     }
+
+    /* Restore umask for minimal side affects */
+    umask(previous_umask); 
 }
 
 static void termReset(void) {
-- 
1.6.1.3

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux