Frage zu Sicherheit einer bestimmten mod Codezeile

[JaneDoe]

Code: Alles auswählen

$start = ( isset($HTTP_GET_VARS['start']) ) ? $HTTP_GET_VARS['start'] : 0;

Das soll eine Lücke im links mod darstellen welche evtl besser durch

Code: Alles auswählen

$start = ( isset($HTTP_GET_VARS['start']) ) ? intval(htmlspecialchars($HTTP_GET_VARS['start'])) : 0;

ersetzt erden soll (Thread bei phpbb.de)

Nun finde ich die obere Zeile in einigen mods und z.B der memberlist.php

Wäre es nun sinnvoll dort auch die vorgeschlagenen Ersetzung zu machen, oder wäre das eine
Einzelfallentscheidung?

[oxpus]

In der memberlist.php ist es original so:

Code: Alles auswählen

$start = ( isset($HTTP_GET_VARS['start']) ) ? intval($HTTP_GET_VARS['start']) : 0;
$start = ($start < 0) ? 0 : $start;

Also ja: Auf jeden Fall alle "veralteten" Fundstellen anpassen.

Und btw:

Code: Alles auswählen

intval(htmlspecialchars($HTTP_GET_VARS['start']))

ist arg übertrieben und unsinnig.

Code: Alles auswählen

intval($HTTP_GET_VARS['start'])

ist mehr als ausreichend.

[JaneDoe]

Danke dir