[PEAR-BUG] Bug #16819 [Opn]: IMAPProtocol::cmdStatus fails on error

Edit report at https://pear.php.net/bugs/bug.php?id=16819&edit=1

ID: 16819
Updated by: daniel.oconnor@gmail.com
Reported By: sam at archives dot org dot au
Summary: IMAPProtocol::cmdStatus fails on error
-Status: Open
+Status: Closed
Type: Bug
Package: Net_IMAP
Operating System: MacOS
Package Version: 1.1.0
PHP Version: 5.2.5
-Assigned To:
+Assigned To: doconnor
Roadmap Versions:
New Comment:

-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: doconnor
This bug has been fixed in SVN.

If this was a documentation problem, the fix will appear on pear.php.net
by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should
be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.




Previous Comments:
------------------------------------------------------------------------

[2012-01-01 23:38:11] doconnor

Thanks for the patch Sam. It's been a while since you uploaded it, and
no reply - so
I'm marking the package unmaintained.

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

[2009-11-22 04:17:46] samwilson

Added #patch
bug:16819;patch:IMAPProtocol_1-1-0.patch;revision:1258863466;.

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

[2009-11-22 04:15:42] samwilson

Description:
------------
The return value of $this->_genericCommand needs to be checked before it
is used as an array (it might be a PEAR_Error).


Test script:
---------------
require_once 'Net/IMAP.php';
$imap = new Net_IMAP('mail.example.org');
$login = $imap->login('username', 'password');
$imap->numMsg();


Expected result:
----------------
[Nothing]

Actual result:
--------------
Fatal error: Cannot use object of type PEAR_Error as array in
/home/user/pear/PEAR/Net/IMAPProtocol.php on line 1038

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