posix: validate mode argument in posix_mkfifo()

Author: arshidkv12

ext/standard/tests/file/filetype_variation.phpt

@@ -47,12 +47,6 @@ posix_mkfifo( $file3, 0755);
 print( filetype( $file3) )."\n";
 unlink($file3);
 
-try {
-    posix_mkfifo('zz.txt', 10000 );
-} catch (\ValueError $e) {
-    echo $e->getMessage() . PHP_EOL;
-}
-
 /* Checking with block in file */
 /* To test this PEAR package should be installed */
 
@@ -71,6 +65,5 @@ link
 dir
 -- Checking with fifo --
 fifo
-posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 07777
 
 *** Done ***

ext/standard/tests/file/posix_mkfifo_invalid_mode.phpt

@@ -0,0 +1,30 @@
+--TEST--
+posix_mkfifo(): invalid mode argument
+--FILE--
+<?php
+
+// Negative mode
+try {
+    posix_mkfifo(__DIR__ . "/testfifo1", -1);
+} catch (ValueError $e) {
+    echo $e->getMessage(), "\n";
+}
+
+// Too large mode
+try {
+    posix_mkfifo(__DIR__ . "/testfifo2", 010000); // > 07777
+} catch (ValueError $e) {
+    echo $e->getMessage(), "\n";
+}
+
+// Garbage bits
+try {
+    posix_mkfifo(__DIR__ . "/testfifo3", 020000); // S_IFCHR bit
+} catch (ValueError $e) {
+    echo $e->getMessage(), "\n";
+}
+?>
+--EXPECTF--
+posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 07777
+posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 07777
+posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 07777