ID:               33500
 Updated by:       paj...@php.net
 Reported By:      ed2019 at columbia dot edu
-Status:           Assigned
+Status:           Closed
 Bug Type:         Feature/Change Request
 Operating System: *
 PHP Version:      5.2.9
 Assigned To:      pajoye
 New Comment:

This bug has been fixed in SVN.

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:
------------------------------------------------------------------------

[2010-01-05 01:12:19] s...@php.net

Automatic comment from SVN on behalf of pajoye
Revision: http://svn.php.net/viewvc/?view=revision&revision=293126
Log: - [doc] add support for DISABLE_AUTHENTICATOR in imap_open (fix
#33500)

------------------------------------------------------------------------

[2010-01-05 01:02:16] s...@php.net

Automatic comment from SVN on behalf of pajoye
Revision: http://svn.php.net/viewvc/?view=revision&revision=293124
Log: - [doc] fix exchange and other imap server support when a
preferred auth method is not desired. Add option support to imap_open.
Only 'DISABLE_AUTHENTICATOR' is supported yet, see #33500 for an example

------------------------------------------------------------------------

[2009-11-04 01:00:01] php-bugs at lists dot php dot net

No feedback was provided for this bug for over a week, so it is
being suspended automatically. If you are able to provide the
information that was originally requested, please do so and change
the status of the bug back to "Open".

------------------------------------------------------------------------

[2009-11-02 16:22:40] nick at mailtrust dot com

It looks like you may have forgotten to add the following to your
patch:

Index: php_imap.c
===================================================================
--- php_imap.c        (revision 3434)
+++ php_imap.c        (working copy)
@@ -105,6 +105,7 @@
ZEND_BEGIN_ARG_INFO_EX(arginfo_imap_open, 0, 0, 3)
       ZEND_ARG_INFO(0, mailbox)
       ZEND_ARG_INFO(0, user)
       ZEND_ARG_INFO(0, password)
       ZEND_ARG_INFO(0, options)
       ZEND_ARG_INFO(0, n_retries)
+      ZEND_ARG_INFO(0, params)
ZEND_END_ARG_INFO()

This should allow for a max of 6 arguments instead of just 5.

------------------------------------------------------------------------

[2009-10-27 11:24:31] paj...@php.net

It sounds like the patch was not applied then. It clearly takes 3 or 6
parameters:
+       if (zend_parse_parameters(argc TSRMLS_CC, "sss|lla", &mailbox,
&mailbox_len, &user, &user_len,
+               &passwd, &passwd_len, &flags, &retries, &params) == FAILURE) {

Can you verify that the patch was actually applied?

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/33500

-- 
Edit this bug report at http://bugs.php.net/?id=33500&edit=1

Reply via email to