[1.7 - 1.7.8] Security fix for CSRF vulnerability

[kai]

A cross-site request forgery vulnerability in 4images 1.7 - 1.7.8 has been found.

To fix this please follow these 4 steps:

1.
Download the attached file csrf_utils.php file and copy it into includes/ folder of your 4images installation.


2.
Open global.php and search for the following line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1	@include(ROOT_PATH.'config.php');

and insert the following code ABOVE this line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1 2 3 4 5 6 7

// Initialize CSRF protection configuration $csrf_protection_enable      = 1; $csrf_protection_frontend    = 1; $csrf_protection_backend     = 1; $csrf_protection_expires     = 7200; $csrf_protection_name        = '__csrf'; $csrf_protection_xhtml       = 1;

In the same file, search for the line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1	include_once(ROOT_PATH.'includes/captcha_utils.php');

and insert the following code BELOW this line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1 2 3 4

//----------------------------------------------------- //--- CSRF protection --------------------------------- //----------------------------------------------------- include_once(ROOT_PATH.'includes/csrf_utils.php');

3.
Open includes/page_header.php and search for the following line (at the end for the file):

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1

?>

and insert the following code ABOVE this line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1 2 3

if ($csrf_protection_enable && $csrf_protection_frontend) {   csrf_start(true); }

4.
Open admin/admin_global.php and search for the following line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1	include_once(ROOT_PATH.'admin/admin_functions.php');

and insert the following code BELOW this line:

PHP Code:  [Select]  [Expand]  [Hide line numbers]

1 2 3

if ($csrf_protection_enable && $csrf_protection_backend) {   csrf_start(); }

Thanks to Russ McRee for finding and reporting this vulnerability!

[Jan-Lukas]

seit ich das eingebaut habe, klappt die Suche nicht mehr von außerhalb der Galerie.
bekomme dann die Fehlermeldung

CSRF check failed.

Zur Erklärung, habe auf einer anderen Seite eine Suchbox installiert, die zu 4images führt, und bis zu diesem update auch geklappt hat.
http://www.ue-ei-portal.de

Die Suche auf 4images klappt natürlich weiterhin

LG

[Rembrandt]

Seit dem Update bekomme ich beim Aufruf folgende Fehlermeldung (wenn keine &sessionid=*** vorhanden ist):
...
Wie kann ich die Fehlermeldung wieder abstellen?

im moment funktioniert deine seite...

ich pers. habe heute bei 4 galerien das "fix" eingebaut und nirgends gab es probleme..

[Hagen-Roderich]

Ja, Seite funktioniert wieder  

Ich habe mir mal die Fehlermeldung durchgelesen und dann die Zeile 249 in der "csrf_utils.php" angeschaut.
Da waren bei mir zwei Leerzeilen hinter "?>".

Danke das Du dir die Zeit dafür genommen hast.

[Jan-Lukas]

das Problem kann ich auch nicht bestätigen, und Session ID's werden bei mir auch nicht angezeigt.
Kannst Du denn mal eine externe suche testen ?

[ivan]

Plugins funktionieren seit Integration nicht mehr!

Migrate Keywords
Clear Cache
Rebuild Search Index
Files Check

Fehler: CSRF check failed.
Erscheint bei drücken des Button "Abschicken"

[Jan-Lukas]

Diese auch nicht, gehe davon aus das keine funktionieren

Image Group
Data Backup
Rebuild Thumbnails
Shoutbox Admin

[Rembrandt]

...
Image Group
...

meinst du meine mod?

[Jan-Lukas]

jep
wenn ich im Admin die ID aufrufe, kommt die Meldung

[Rembrandt]

ahjo.. so ein schmorn...
ich habe in der global.php das mal auf "0" gesetzt:

Code:  [Select]  [Expand]  [Hide line numbers]

1	$csrf_protection_backend     = 0;

aber es wird sicher noch eine lösung dafür geben...

mfg Andi

[Jan-Lukas]

damit klappen die internen, bei externen (meine suche)
habe ich noch $csrf_protection_frontend    = 0; auf null gestellt.
Hoffe das kai was findet

[x23piracy]

ahjo.. so ein schmorn...
ich habe in der global.php das mal auf "0" gesetzt:

Code:  [Select]  [Expand]  [Hide line numbers]

1	$csrf_protection_backend     = 0;

aber es wird sicher noch eine lösung dafür geben...

mfg Andi

Hi Andi,

schaltet das den ganzen Schutz ab?


Gruß Jens

[Jan-Lukas]

nein, wenn du das auf 0 stellst, ist alles aus

$csrf_protection_enable      = 1;

[Jan]

Ich habe die csrf_utils.php im Hauptpost aktualisiert. Bitte ersetzt die aktuelle mit dieser, dann sollten auch die Plugins wieder funktionieren.

Zu den Konfigurationsvariablen:

Code:  [Select]  [Expand]  [Hide line numbers]

1	$csrf_protection_enable      = 1;

Dies auf 0 gesetzt, schaltet den Schutz komplett aus (für Frontend und Backend).

Code:  [Select]  [Expand]  [Hide line numbers]

1	$csrf_protection_frontend    = 1;

Dies auf 0 gesetzt, schaltet den Schutz komplett für das Frontend aus.

Code:  [Select]  [Expand]  [Hide line numbers]

1	$csrf_protection_backend     = 1;

Dies auf 0 gesetzt, schaltet den Schutz komplett für das Backend aus.

Meine Empfehlung:

$csrf_protection_enable und $csrf_protection_backend sollten in jedem Fall auf 1 gesetzt sein. Falls es Probleme gibt, bitte hier melden. Im Falle von Problemen kann man $csrf_protection_backend temporär auf 0 setzen um benötigte Änderungen durchzuführen. Danach wieder auf 1 setzen.

$csrf_protection_frontend sollte man natürlich am besten auf 1 belassen. Falls es Probleme gibt, kann man hier aber vorrübergehend 0 setzen um die Galerie am Laufen zu halten.

[Rembrandt]

Ich habe die csrf_utils.php im Hauptpost aktualisiert. Bitte ersetzt die aktuelle mit dieser, dann sollten auch die Plugins wieder funktionieren.
...

funktioniert tatsächlich  danke!

mfg Andi