diff options
| author | Richard Jones <rjones@redhat.com> | 2010-04-17 13:33:14 +0100 |
|---|---|---|
| committer | Richard Jones <rjones@redhat.com> | 2010-04-17 13:36:26 +0100 |
| commit | 61ab83d19009a8006dd73ebe16d22494b78be4d1 (patch) | |
| tree | ad14a0f005867ceea391da618792c59c9b395728 /daemon/umask.c | |
| parent | 42fc0e02b094ef3b19dae115276a0a33116af6d5 (diff) | |
| download | libguestfs-61ab83d19009a8006dd73ebe16d22494b78be4d1.tar.gz libguestfs-61ab83d19009a8006dd73ebe16d22494b78be4d1.tar.xz libguestfs-61ab83d19009a8006dd73ebe16d22494b78be4d1.zip | |
Improved checking, documentation of modes (RHBZ#582901, RHBZ#582929).
chmod: Disallow negative mode, document mode affected by umask.
mkdir-mode: Disallow negative mode, document that filesystems
may interpret the mode in different ways.
mknod: Disallow negative mode, document mode affected by umask.
umask: Check the range of umask mask value carefully.
Diffstat (limited to 'daemon/umask.c')
| -rw-r--r-- | daemon/umask.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/daemon/umask.c b/daemon/umask.c index a9ddeab0..db4a2f2f 100644 --- a/daemon/umask.c +++ b/daemon/umask.c @@ -35,6 +35,11 @@ do_umask (int mask) { int r; + if (mask < 0 || mask > 0777) { + reply_with_error ("0%o: mask negative or out of range", mask); + return -1; + } + r = umask (mask); if (r == -1) { |