From: Felipe Pena <felipe@php.net>
Date: Sun, 19 Jun 2011 14:27:33 +0000 (+0000)
Subject: - Added missing void param check in sys_get_temp_dir
X-Git-Tag: php-5.5.0alpha1~1910
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=54d5662919f9490ba19566de952272895a7483ea;p=php

- 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 6748d326a7..3fe4ef69ea 100644
--- 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);
 }
 /* }}} */