php/archived-php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2026-03-24 00:02:20 +01:00
Code Issues Packages Projects Releases Wiki Activity

Revert "ext/posix: validate permissions argument in posix_mkfifo()"

Browse Source 
This reverts commit 6bd97e72b7.
This commit is contained in:
David Carlier
2026-02-16 17:49:37 +00:00
parent 99bf26be0b
commit b3c259c13c
3 changed files with 0 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
NEWS
View File

@@ -80,8 +80,6 @@ PHP NEWS
- Posix:
. Added validity check to the flags argument for posix_access(). (arshidkv12)
. Added validity check to the permissions argument for posix_mkfifo().
(arshidkv12)
- Reflection:
. Fixed bug GH-20217 (ReflectionClass::isIterable() incorrectly returns true

5
ext/posix/posix.c
View File

@@ -621,11 +621,6 @@ PHP_FUNCTION(posix_mkfifo)
RETURN_FALSE;
}
if (mode < 0 || (mode & ~07777)) {
zend_argument_value_error(2, "must be between 0 and 0o7777");
RETURN_THROWS();
}
result = mkfifo(ZSTR_VAL(path), mode);
if (result < 0) {
POSIX_G(last_error) = errno;

36
ext/posix/tests/posix_mkfifo_invalid_mode.phpt
View File

@@ -1,36 +0,0 @@
--TEST--
posix_mkfifo(): invalid mode argument
--SKIPIF--
<?php
if (!function_exists("posix_mkfifo")) {
die("skip no posix_mkfifo()");
}
?>
--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 0o7777
posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 0o7777
posix_mkfifo(): Argument #2 ($permissions) must be between 0 and 0o7777