Patchworkβ umask in device node creation error handling

login
register
about
Submitter Florian Zumbiehl
Date 2009-08-26 15:51:17
Message ID <20090826155117.GU2717@florz.florz.dyndns.org>
Download mbox | patch
Permalink /patch/44056/
State New
Headers show

Comments

Florian Zumbiehl - 2009-08-26 15:51:17
Hi,

I guess that the diff is pretty much self-explaining ...


Florian

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Patch

diff --git a/libdm/libdm-common.c b/libdm/libdm-common.c
index b7782a7..554c1a4 100644
--- a/libdm/libdm-common.c
+++ b/libdm/libdm-common.c
@@ -360,6 +360,7 @@  static int _add_dev_node(const char *dev_name, uint32_t major, uint32_t minor,
 
 	old_mask = umask(0);
 	if (mknod(path, S_IFBLK | mode, dev) < 0) {
+		umask(old_mask);
 		log_error("Unable to make device node for '%s'", dev_name);
 		return 0;
 	}