- Added missing void param check in sys_get_temp_dir

- Fixed param check of umask()

diff --git a/ext/standard/file.c b/ext/standard/file.c
index 6748d326a7aec1e426da57aad1f01f3bec86605d..3fe4ef69ea510af6ca950efe6f45e61a67e22dec 100644 (file)
--- a/ext/standard/file.c
+++ b/ext/standard/file.c
@@ -1413,7 +1413,10 @@ PHP_FUNCTION(umask)
 {
        long arg1 = 0;
        int oldumask;
-       int arg_count = ZEND_NUM_ARGS();
+       
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &arg1) == FAILURE) {
+               RETURN_FALSE;
+       }
 
        oldumask = umask(077);
 
@@ -1421,12 +1424,9 @@ PHP_FUNCTION(umask)
                BG(umask) = oldumask;
        }
 
-       if (arg_count == 0) {
+       if (ZEND_NUM_ARGS() == 0) {
                umask(oldumask);
        } else {
-               if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &arg1) == FAILURE) {
-                       RETURN_FALSE;
-               }
                umask(arg1);
        }
 
@@ -2491,6 +2491,9 @@ PHP_FUNCTION(fnmatch)
    Returns directory path used for temporary files */
 PHP_FUNCTION(sys_get_temp_dir)
 {
+       if (zend_parse_parameters_none() == FAILURE) {
+               return;
+       }
        RETURN_STRING((char *)php_get_temporary_directory(), 1);
 }
 /* }}} */