#29753 [Opn->Csd]: Parameters of mcal_fetch_event()

ID: 29753 Updated by: iliaa@php.net Reported By: vrana@php.net -Status: Open +Status: Closed Bug Type: MCAL related Operating System: Irrelevant PHP Version: 5.0.1 New Comment: This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2004-08-19 09:20:20] vrana@php.net Description: ------------ Proto for mcal_fetch_event() says it accepts 2 or 3 parameters but code says it accepts 1, 2 or 3 parameters. I belive proto is right in this case and I am sending a patch for it. Patch proposal: RCS file: /repository/pecl/mcal/php_mcal.c,v retrieving revision 1.59 diff -u -r1.59 php_mcal.c --- mcal/php_mcal.c 16 Jun 2003 14:03:30 -0000 1.59 +++ mcal/php_mcal.c 18 Aug 2004 12:02:38 -0000 @@ -459,7 +459,7 @@ CALEVENT *myevent; int myargcount=ZEND_NUM_ARGS(); - if (myargcount < 1 || myargcount > 3 || zend_get_parameters_ex(myargcount, &streamind, &eventid, &options) == FAILURE) { + if (myargcount < 2 || myargcount > 3 || zend_get_parameters_ex(myargcount, &streamind, &eventid, &options) == FAILURE) { WRONG_PARAM_COUNT; } convert_to_long_ex(streamind); ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=29753&edit=1