Menu

#1600 Reply Error(Unable to find document) Squirrelmail 1.4.3 Stbl

closed-fixed
nobody
Compose (426)
5
2004-06-10
2004-05-31
Anonymous
No

USING NEW STABLE Squirrelmail 1.4.3
Apache 1.3.29 (SSL)

Situation :

I've read an E-mail and press the Reply button.
I get an error that the connection has ended while
executing the script.

The direct URL :
https://82.161.5.209/mail/src/compose.php?
passed_id=1&mailbox=INBOX&startMessage=1&passed_en
t_id=0&smaction=forward

Apache produces the following error :
Allowed memory size of 83886080 bytes exhausted (tried
to allocate 79585414 bytes).

(It looks like PHP enters some kind of loop while entering
the reply request)

Increasing memory_limit from 80Mb!! to 180 Mb results
in :
Allowed memory size of 188743680 bytes exhausted
(tried to allocate 159170870 bytes)

Other things you need 2 know :

Forwarding a message DOES work :
https://82.161.5.209/mail/src/compose.php?
passed_id=1&mailbox=INBOX&startMessage=1&passed_en
t_id=0&smaction=forward

Discussion

  • Nobody/Anonymous

    Logged In: NO

    I found out in COMPOSE.PHP the endless loop is probably the
    foreach ($entities as $ent) {
    (Line 613)

    The old COMPOSE.PHP (SM 1.4.2) works ok though.

     
  • Nobody/Anonymous

    Logged In: NO

    I am having the same problem with php 4.3.6 and apache2.0.49
    The 1.4.3 is not workable this way it consumes al the memory
    and only crashes the system..

    Is this really a stable version?

     
  • rozeel

    rozeel - 2004-05-31

    Logged In: YES
    user_id=1053626

    I am having the same problem with php 4.3.6 and apache2.0.49
    The 1.4.3 is not workable this way it consumes al the memory
    and only crashes the system..

    Is this really a stable version?

    replacing the compose.php from 1.4.2 did a hell of a job

    a same that this was not tested...

    Thanks..

     
  • Tomas Kuliavas

    Tomas Kuliavas - 2004-05-31

    Logged In: YES
    user_id=225877

    Making a better report. Problem can be reproduced if you
    have multipart message.

    I think experimental option accidently commited to stable
    was not reverted correctly.

    attached diff between 1.319.2.34 and 1.319.2.32.

     
  • Tomas Kuliavas

    Tomas Kuliavas - 2004-05-31
     
  • Genotix

    Genotix - 2004-05-31

    Logged In: YES
    user_id=1053513

    Signed me up for sourceforge now :)

    One thing seems to lack the old COMPOSE.PHP (The one from
    SM 1.4.2) that's the address book.

    I'm unable to use it at the moment. :(

     
  • Tomas Kuliavas

    Tomas Kuliavas - 2004-05-31
    • status: open --> open-fixed
     
  • Tomas Kuliavas

    Tomas Kuliavas - 2004-05-31

    Logged In: YES
    user_id=225877

    Problem should be fixed in 1.4.4cvs. Use attached patch or
    download compose.php v.1.319.2.35/1.319.2.32 from cvs.
    Please note that anonymous cvs uses backup server with 5
    hour difference. You might be unable to get 1.319.2.35 right
    now.

    Sorry for all the trouble caused by this typo.

    Keeping bug open until fixed package is released.

     
  • Genotix

    Genotix - 2004-05-31

    Logged In: YES
    user_id=1053513

    Sorry, tokul, didn't read your message yet.

    The diff solves the problem.

    Great job :)

    For people unfimiliar with .diff files :

    go to line 760 of [SMDIR]/src/compose.php
    From that point delete 4 lines and insert lines below.

    $body .= '>' . str_replace("\n", "\n>$gt ", rtrim
    ($rewrap_body[$i])) ."\n";
    } else {
    $body .= '> ' . str_replace("\n", "\n> ", rtrim
    ($rewrap_body[$i])) . "\n";
    }

     
  • Jaap Grol

    Jaap Grol - 2004-05-31

    Logged In: YES
    user_id=296387

    Same here, regardless of browser brand.
    Emty compose windows works fine as well.

     
  • Nobody/Anonymous

    Logged In: NO

    Looks like someone made a mistake while cleaning up some
    code.
    $body .= $body .= '>' . str_replac.... is causing the problem.
    This should be $body .= '>' . str_replac....

    Here a patch:

    --- ./compose.old 2004-05-24 09:56:51.000000000
    +0200
    +++ ./compose.php 2004-06-01 14:43:54.000000000
    +0200
    @@ -757,9 +757,9 @@
    sqWordWrap($rewrap_body[$i], ($editor_size));
    if (preg_match("/^(>+)/", $rewrap_body[$i],
    $matches)) {
    $gt = $matches[1];
    - $body .= $body .= '>' . str_replace
    ("\n", "\n>$gt ",rtrim($rewrap_body[$i])) ."\n";
    + $body .= '>' . str_replace("\n", "\n>$gt ",rtrim
    ($rewrap_body[$i])) ."\n";
    } else {
    - $body .= $body .= '> ' . str_replace
    ("\n", "\n> ",rtrim($rewrap_body[$i])) . "\n";
    + $body .= '> ' . str_replace("\n", "\n> ",rtrim
    ($rewrap_body[$i])) . "\n";
    }
    unset($rewrap_body[$i]);
    }

     
  • Thijs Kinkhorst

    Thijs Kinkhorst - 2004-06-02

    Logged In: YES
    user_id=285765

    This has been fixed in SquirrelMail Stable CVS and an
    updated version will be released soon.

     
  • Paul Lesniewski

    Paul Lesniewski - 2004-06-10
    • status: open-fixed --> closed-fixed
     
  • Paul Lesniewski

    Paul Lesniewski - 2004-06-10

    Logged In: YES
    user_id=508228

    This issue is resolved in release 1.4.3a.

     

Log in to post a comment.