[Tickets #11671] Re: The dynamic view crashes on some Inboxes

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/11671
------------------------------------------------------------------------------
  Ticket             | 11671
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | The dynamic view crashes on some Inboxes
  Queue              | IMP
  Version            | 6.0.1
  Type               | Bug
  State              | Not A Bug
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2012-11-29 11:24) wrote:

>
>> PHP is seriously broken if this somehow causes a segfault
>>
> From the PHP documentation
> "PCRE's recursion limit. Please note that if you set this value to a  
> high number you may consume all the available process stack and  
> eventually crash PHP (due to reaching the stack size limit imposed  
> by the Operating System)."

That should be up to the programmer to determine.  I don't want the  
programming language enforcing arbitrary limits.

If a programmer wants to shoot themselves in the foot - that is their  
prerogative.  But this limit makes certain tasks impossible,  
especially when the input data size is unknown (say, for example, an  
e-mail message where the PHP programmer has no control over limiting  
this input size).

>> 5000 is **WAY** too low.  That value is going to cause all sorts of
>> issues with any sort of even moderately complex regex that contains
>> backreferences.  For that matter, 100000 is also arguably too low.
>
> Well 10000 makes IMP > 6.0 crash, so I'll have a chat with the  
> sysadmin to see if he can find out why this is happening.

There is something wrong with your system then.  Because I can run  
with this number at 10,000,000 and nothing happens.