[Tickets #12160] Infinity loop on event with recurrence

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12160
------------------------------------------------------------------------------
  Ticket             | 12160
  Created By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Infinity loop on event with recurrence
  Queue              | Kronolith
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-04-05 15:58) wrote:

Hi,

I've traced an infinity loop with kronolith for two days now.
Neither xdebug nor xhprof were of any help since
the script never finished or timed out if PHP limits were active.

The root problem is that kronolith/lib/kronolith.php::addEvents() does not
check the return value of "$event->recurrence->nextRecurrence()" in two
places and just assumes it's a Horde_Date object afterwards.
This is wrong as nextRecurrence() might return 'false'

The problem itself also does not show with the kronolith web UI,
it's only triggered when using ActiveSync. Probably ActiveSync
sets a "funny" startDate or endDate for the addEvents() call.

Please see the attached patch how I modified the code
to output more debug information + the actual debug log.

Cheers,
Thomas