php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #29753 Parameters of mcal_fetch_event()
Submitted: 2004-08-19 09:20 UTC Modified: 2007-04-04 18:51 UTC
From: vrana@php.net Assigned:
Status: Closed Package: Unknown/Other Function
PHP Version: 5.0.1 OS: Irrelevant
Private report: No CVE-ID: None
 [2004-08-19 09:20 UTC] 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);



Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2004-08-20 02:46 UTC] iliaa@php.net
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.


 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Mar 28 22:01:26 2024 UTC