Menü
aus ein
Ansicht umschalten Baum an
Avatar von Oliver K.
  • Oliver K.

10 Beiträge seit 26.03.2002

Erleuchtet mich...

..als jemand, der sonst mit "sicheren" Programmiersprachen umgeht
(*hust*), oder die zumindest denjenigen Sprachen, die die derben,
potentiellen Hämmer aus C nebst Standardbibliotheken nicht so
aufweisen, muss ich mal ganz unverblümt fragen:

Wo war denn jetzt was beim 1. Patch
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/string.c?r1=1.445.
2.14.2.57&r2=1.445.2.14.2.58

genau verkehrt?

VOR dem Patch ist mir der Fehler klar... Dass die darauf folgenden
Fixes irgendwie ... "seltsam" sind habe ich auch im Gefühl.

Aber was war denn beim 1. ?

Okay, das mit

if (out_len > INT_MAX || (....)

ist wohl quark, weil an der Stelle wohl schon zu spät.
Aber out_len schlägt doch bei einer überschreitung des Wertebereichs
ohnehin
in den negativen Bereich um (zumindest bei mir hier.. wenn ich analog
mir das nachbaue..) , und wird dann von der Bedingung

(....) || out_len <= 0)

doch abgefangen?

Klar, dem ist wohl nicht so, aber kann mir jemand die Tomaten von den
Augen entfernen? (Bin halt nicht C-abgehärtet..)

Gruß

Bewerten
- +
Anzeige