From 5357e24abca0d493d8ff7b1f691bb661db814116 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Thu, 3 Mar 2011 13:38:52 +0000 Subject: [PATCH] 2011-03-03 Chris Johns PR 1749/filesystem * libcsupport/src/mknod.c: Fix the incorrect handling of the file type in the mode value o reject invalid types as per the standard. --- cpukit/ChangeLog | 6 ++++++ cpukit/libcsupport/src/mknod.c | 15 +++++++++++++-- 2 files changed, 19 insertions(+), 2 deletions(-) diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog index 0d44529c98..6e2e393ce3 100644 --- a/cpukit/ChangeLog +++ b/cpukit/ChangeLog @@ -1,3 +1,9 @@ +2011-03-03 Chris Johns + + PR 1749/filesystem + * libcsupport/src/mknod.c: Fix the incorrect handling of the file type + in the mode value o reject invalid types as per the standard. + 2011-01-21 Eric Norum * libmisc/capture/capture.c: Avoid using TCB of task just deleted. diff --git a/cpukit/libcsupport/src/mknod.c b/cpukit/libcsupport/src/mknod.c index c786c6d756..c2c86e7afe 100644 --- a/cpukit/libcsupport/src/mknod.c +++ b/cpukit/libcsupport/src/mknod.c @@ -40,8 +40,19 @@ int mknod( const char *name_start; int result; - if ( !(mode & (S_IFREG|S_IFCHR|S_IFBLK|S_IFIFO) ) ) - rtems_set_errno_and_return_minus_one( EINVAL ); + /* + * The file type is field within the mode. Check we have a sane mode set. + */ + switch (mode & S_IFMT) { + case S_IFDIR: + case S_IFCHR: + case S_IFBLK: + case S_IFREG: + case S_IFIFO: + break; + default: + rtems_set_errno_and_return_minus_one( EINVAL ); + } if ( S_ISFIFO(mode) ) rtems_set_errno_and_return_minus_one( ENOTSUP );