diff options
| author | Rasmus Lerdorf <rasmus@php.net> | 2001-08-13 06:43:47 +0000 |
|---|---|---|
| committer | Rasmus Lerdorf <rasmus@php.net> | 2001-08-13 06:43:47 +0000 |
| commit | 8c497f05c4d93f4054f67fb9875f0894364020e2 (patch) | |
| tree | 0c3a97abb0de616fb036f38e18f752e042b36623 /ext/mhash | |
| parent | 3f505f8624836b32f919b9fbcc533b8e069a87c9 (diff) | |
| download | php-git-8c497f05c4d93f4054f67fb9875f0894364020e2.tar.gz | |
We don't consistently check for args passed to functions that don't
take any args. In some cases we probably want to skip the check for
performance reasons, but in other cases where performance is unlikely
to be a factor, not throwing a warning on the wrong number of args passed
to a function is at best inconsistent, and at worst it could hide a bug.
So, add a few such checks. There are still lots of cases out there.
Diffstat (limited to 'ext/mhash')
| -rw-r--r-- | ext/mhash/mhash.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ext/mhash/mhash.c b/ext/mhash/mhash.c index e630b7c392..d6ad84c7cf 100644 --- a/ext/mhash/mhash.c +++ b/ext/mhash/mhash.c @@ -83,6 +83,9 @@ static PHP_MINIT_FUNCTION(mhash) Gets the number of available hashes */ PHP_FUNCTION(mhash_count) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "") == FAILURE) + return; + RETURN_LONG(mhash_count()); } |