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/interbase | |
| 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/interbase')
| -rw-r--r-- | ext/interbase/interbase.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ext/interbase/interbase.c b/ext/interbase/interbase.c index 273520c58f..0ed2c2412a 100644 --- a/ext/interbase/interbase.c +++ b/ext/interbase/interbase.c @@ -236,6 +236,9 @@ typedef struct { Return error message */ PHP_FUNCTION(ibase_errmsg) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "") == FAILURE) + return; + if (IBG(errmsg[0])) { RETURN_STRING(IBG(errmsg), 1); } |