Seit den letzten Changesets teilweise weisse Seite nach Login

[Torsten Riemer]

Ich weiss nicht genau, ob es mit den Änderungen für die Umbenennung des "/admin/" Ordners zu tun hat oder mit der Einführung der "paths.php" oder mit der Session-Überarbeitung. Jedenfalls erhalte ich nach Login öfters eine weisse Seite.

Es wäre vielleicht für die nächste Zeit nicht verkehrt das "error_reporting" in den Demoshops dauerhaft zu aktivieren, damit ich die Fehlermeldung weitergeben kann.

[Gerhard Waldemair]

ich habe das error_reporting für den 1.06er Demoshop auf

error_reporting(E_ALL & ~E_NOTICE & ~E_STRICT);

gestellt.

[Torsten Riemer]

Danke dir. Jetzt muss der Fehler nur nochmal auftreten. ;-)

[Tekin Birdüzen]

Ich finde zur Entwicklung sollte das & ~E_NOTICE entfernt werden, denn auch Notices können die Funktion beeinträchtigen. In der Goldversion kann dies dann wieder rein um sicherzustellen das schlecht programmierte Module nicht Störungen verursachen.

[Gerhard Waldemair]

So, habe das error_reporting umgestellt.

Hier meine Fehlermeldung. Diese habe ich aber früher schon erhalten, also nicht erst seit der Änderungen am Sessionhandling.

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 3575275257 bytes) in /......./tags/modified-1.06/inc/xtc_href_link.inc.php on line 61

Was also verursacht hier über 3GB Speicher:

if (defined('SID') && xtc_not_null(SID)) {

[Torsten Riemer]

Jupp, konnte ich jetzt auch mehrfach verfolgen, wobei es bei mir nicht gleich 3GB waren. ;-)

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 685866745 bytes) in /var/www/clients/client1/web71/web/tags/modified-1.06/inc/xtc_href_link.inc.php on line 61

[H. H. H.]

Wie wurde das Problem gefixt?
Wollte es im Testshop nachstellen, doch da war es schon behoben.

[Gerhard Waldemair]

noch nicht gelöst. Das Problem ist nicht immer nachstellbar. Ich konnte es mehrfach nachstellen indem ich Browsercache, Cookies, Temporäre Files, etc gelöscht hatte. Shop aufgerufen und angemeldet.

Ich habe schon den Verdacht, dass es mit dem 5 minütlichen SVN Update Intervall zu tun hat.

[Torsten Riemer]

Um es noch interessanter zu machen hatte ich auch teilweise einen 500er über die .htaccess (ErrorDocument 500 /sitemap.html?error=500) bzw sitemap.html:

Fehler 500: Beim Server gab es einen internen Fehler.

[Simon Heinrich]

ZenCart hat wohl das gleiche Problem mit den Sessions, siehe
​http://www.zen-cart.com/showthread.php?207435-php-5-4-fatal-error-Allowed-memory-size-of-83886080-bytes-exhausted

Sie tippen auf eine Endlosschleife, wenn ich das richtig verstanden habe...

[Torsten Riemer]

Franky schrieb mir dazu gerade, dass er einen Fehler in der "/inc/xtc_href_link.inc.php" in Z.56 vermutet:

    while ( (substr($link, -1) == '&') || (substr($link, -1) == '?') ) {
      $link = substr($link, 0, -1);
    }

sollte sein:

    if ( (substr($link, -1) == '&') || (substr($link, -1) == '?') ) {
      $link = substr($link, 0, -1);
    }

Wenn dem so ist, dann haben wir hier den Speicher-Killer gefunden.

Analog auch in der "/inc/xtc_href_link_from_admin.inc.php".

[H. H. H.]

$link = rtrim($link, '&?'); würde hier auch ausreichen und hat noch den selben Effekt welchen das while haben sollte(alle entsprechenden Zeichen am Ende entfernen).

Wobei an der Stelle dieser Code eigentlich nicht benötigt wird, wenn die Parameter sauber übergeben werden.

[Torsten Riemer]

Ich konnte den Fehler übrigens eben auch im Demoshop 1.06 rev 4642 nachvollziehen. Zuerst erhielt ich wieder:

Fehler 500: Beim Server gab es einen internen Fehler.

und beim reload der Seite dann:

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 3210280697 bytes) in /var/www/clients/client1/web71/web/tags/modified-1.06/inc/xtc_href_link.inc.php on line 62

Nachtrag: Bitte die Changesets r5614 & r5615 von h-h-h genau prüfen und hier diskutieren.

[Torsten Riemer]

Ich habe die Änderungen von h-h-h nach Rücksprache mit Gerhard in r5620 in den Trunk übernommen.

[Tekin Birdüzen]

Das kann aber nicht die Lösung des Fehlers sein, denn in einer Schleife würde jedes Mal ein kleines Stückchen RAM reserviert werden und nicht auf einen Schlag mehr als 3GiB. Die Zahl bei der Reservierung ist jenseits von Gut und Böse.

[H. H. H.]

Problem konnte lokal mit xammp PHP Version 5.4.19 nicht nachgestellt werden.

[Torsten Riemer]

Gerhard hat das Problem jetzt evtl. mit r5705 behoben.

Ich habe es bereits in r5706 in den trunk übernommen.

Bitte haltet mal die Augen auf, ob der Fehler jetzt nochmal auftritt.

[H. H. H.]

Wow, wie ist Gerhard darauf gekommen?
Bei mir Funktioniert es jetzt scheinbar ohne Fehler.

[Tekin Birdüzen]

Gerhard hatte ein längeres Telefonat mit mir wo wir das Thema besprachen und ich Ideen hatte wie man die "böse Stelle" am besten lokalisieren kann. ;) Komisch nur das wir beide davon ausgingen, dass defined das Problem erzeugt und nicht der Funktionsaufruf.

Es ist aber wohl Fakt, das es ein Memory Leak von PHP 5.4 ist. Denke mal es wird in den kommenden Versionen von PHP behoben.

[Torsten Riemer]

Er hatte für's debugging die Zeile in r5680 gesplittet und somit konnten wir genauer sagen, wo der Fehler liegen könnte. Dadurch hat sich der Fehler in Zeile 62 verlagert und wurde mit r5710 scheinbar korrigiert.

[H. H. H.]

Fehler ist noch vorhanden gewesen:
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 3601280761 bytes) in /var/www/clients/client1/web71/web/tags/modified-1.06-dev/inc/xtc_href_link.inc.php on line 59

Habe es jetzt mal mit constant('SID') != '' statt SID != '' versucht um das Problem zu umgehen. Bin gespannt ob es den Memory Leak jetzt noch gibt. ( r5717 )

[Gerhard Waldemair]

ich habe auf dem Server mal das neueste PHP5.4 compiliert.

Version ist jetzt 5.4.19

[H. H. H.]

hattest du vorher geprüft, ob der Fehler noch vorhanden ist?
Hatte nach der Änderung den Memory Leak nicht mehr nachstellen können, schien mir so als ob constant('SID') die Lösung ist.

[Torsten Riemer]

Gerhard hat wieder die alte Version 5.4.4 aktiviert, damit wir schauen können, ob der Fehler durch constant() behoben ist.

[Tekin Birdüzen]

Es könnte dieser Bug gewesen sein, der in Version 5.4.16 behoben wurde:

• Fixed bug ​#64660 (Segfault on memory exhaustion within function definition).

Netterweise ist die Beschreibung dieses Bugs auf private gesetzt und man kann es nicht einsehen.

[Torsten Riemer]

Gerade aktuell wieder erhalten:

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 1199922937 bytes) in /var/www/clients/client1/web71/web/tags/modified-1.06/inc/xtc_href_link.inc.php on line 62

Also der Fix mit constant() hat das Problem auch nicht lösen können.

[Tekin Birdüzen]

Mit der alten oder neuen PHP Version? Wenn es die alte ist und der Fehler wie von mir geschrieben in 5.4.16 behoben wurde im Core können wir eh nicht viel dagegen tun.

[Gerhard Waldemair]

es ist seit heute 9 Uhr wieder PHP Version 5.4.4-14+deb7u4, also das original aus der Debian Wheezy Distri in Verwendung.

[Torsten Riemer]

Bitte schaut nochmal hier: ​Demoshop geht nicht

[Gerhard Waldemair]

Der Fehler kann in der 2.00 nicht mehr nachgestellt werden.