[PHP-DEV] safe_mode.c: php_checkuid() mode
Rasmus, can you clarify the 'mode' parameter in php_checkuid()? In safe_mode.h there are 5 defined values, the 5th is: (which is used in fopen_wrappers.c ...) #define CHECKUID_CHECK_MODE_PARAM 4 However, in safe_mode.c CHECKUID_CHECK_MODE_PARAM is not used, and furthermore if 'fopen_mode' is not null, 'mode' is overwritten anyway (depending if fopen_mode ~= 'r') [line 56ish]. It seems that it exists there for possible expansion ... is this correct? So my question is ... Can/should the 'mode' and the CHECKUID_* defines be changed to bit flags (like php_fopen_wrapper() mode flags)? My motivation for this is to be able to check *just* for a files existence, not checking UID/GID. Possibly a better way to do this would be to split php_checkuid() into php_checkexists() and php_checkuid(), where the former would be called by the latter (as to not break anything)... This might be more intuitive, and then I won't have to duplcate the code to check for the file (as small as it is). Any comments? -James -- PHP Development Mailing List http://www.php.net/ To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]
Re: [PHP-DEV] safe_mode.c: php_checkuid() mode
Nix those mumblings about file existence ... but what about bits vs not-bits for the mode? -James On Mon, 9 Jul 2001, James E. Flemer wrote: Rasmus, can you clarify the 'mode' parameter in php_checkuid()? In safe_mode.h there are 5 defined values, the 5th is: (which is used in fopen_wrappers.c ...) #define CHECKUID_CHECK_MODE_PARAM 4 However, in safe_mode.c CHECKUID_CHECK_MODE_PARAM is not used, and furthermore if 'fopen_mode' is not null, 'mode' is overwritten anyway (depending if fopen_mode ~= 'r') [line 56ish]. It seems that it exists there for possible expansion ... is this correct? So my question is ... Can/should the 'mode' and the CHECKUID_* defines be changed to bit flags (like php_fopen_wrapper() mode flags)? My motivation for this is to be able to check *just* for a files existence, not checking UID/GID. Possibly a better way to do this would be to split php_checkuid() into php_checkexists() and php_checkuid(), where the former would be called by the latter (as to not break anything)... This might be more intuitive, and then I won't have to duplcate the code to check for the file (as small as it is). Any comments? -James -- PHP Development Mailing List http://www.php.net/ To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]
Re: [PHP-DEV] safe_mode.c: php_checkuid() mode
Nix those mumblings about file existence ... but what about bits vs not-bits for the mode? bits vs. non-bits? I'm afraid you have lost me. To get a CVS account, please fill out the little form here: http://php.net/cvs-php.php -Rasmus -- PHP Development Mailing List http://www.php.net/ To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]