[Tickets #13187] Imap_Client: Prevent endless loop in writeLiteral()

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/13187
------------------------------------------------------------------------------
  Ticket             | 13187
  Created By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Imap_Client: Prevent endless loop in writeLiteral()
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2014-05-15 11:47) wrote:

writeLiteral() was stuck in an endless loop
logging this message until the HDD was full:

PHP ERROR: feof() expects parameter 1 to be resource, integer given  
[pid 24223 on line 106 of  
"/usr/share/pear/Horde/Imap/Client/Socket/Connection/Socket.php"]
PHP ERROR: fread() expects parameter 1 to be resource, integer given  
[pid 24223 on line 107 of  
"/usr/share/pear/Horde/Imap/Client/Socket/Connection/Socket.php"]

Prevent it by checking the return value of fread().
While we are at it, check the return value of rewind(), too.

The real cause of this problem is located somewhere else,
but getting stuck in an endless loop isn't nice either :)

Cheers,
Thomas




Thomas Jarosch <thomas.jarosch at intra2net.com> (2014-05-15 11:47)  
uploaded: 0001-Prevent-endless-loop-in-writeLiteral.patch

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=0001-Prevent-endless-loop-in-writeLiteral.patch&ticket=13187&fn=%2F0001-Prevent-endless-loop-in-writeLiteral.patch