---
title: "Nach SecurityUpdate Fehler im ve_guestbook"
url: "https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook"
source: "typo3.net"
tags: ["Extensions","Frage","Update & Upgrade"]
created: "2009-06-16"
last_reply: "2009-06-16"
replies: 6
solved: false
page: 1
pages: 1
lang: "de"
---

# Nach SecurityUpdate Fehler im ve_guestbook

Extensions · 6 Antworten · gestartet 2009-06-16

## Eröffnungsbeitrag

**DiePicknickerin** · 2009-06-16 · [Beitrag #1](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/1)

Hallo,

ich habe gerade nach dem Announce ein Gästebuch upgedated auf 2.7.2 von der Vorversion.

Nun habe ich einen Fehler:

Warning: Invalid argument supplied for foreach() in /xxx/typo3conf/ext/ve_guestbook/pi1/class.tx_veguestbook_pi1.php on line 694

Die Zeile 694 ist das besagte removeBadHTML - das war doch eigentlich der Auslöser für das Update, oder?

Was kann man da tun?

## Antworten

### [#2](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/2) · daimos · 2009-06-16

Stehe gerade vor dem gleichen Problem, bisher noch ohne Lösung.
Die Meldung erscheint nur der Seite, in der man einen neuen Eintrag vornehmen kann.
Es wird auch kein neuer mehr eingetragen.

### [#3](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/3) · DiePicknickerin · 2009-06-16

Mhm, ich habe die foreach-Anweisung jetzt mal auskommentiert und damit die Sicherheitslücke wieder geöffnet ;-)

Immerhin wird ein paar Zeilen tiefer ja wieder strtoupper ud stipslashes ausgeführt.

Nach einer gescheiteren Lösung muss ich noch suchen.

### [#4](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/4) · Norman2k · 2009-06-16

Das liegt daran, dass das Array ($this->postvars) leer ist. Checkt einfach vorher, ob $this->postvars ein Array ist.
Solange keine Werte vom Formular übergeben werden, muss ja auch nichts geprüft werden.

```php
<?php
$this->postvars = t3lib_div::GPvar('tx_veguestbook_pi1');

if(is_array($this->postvars)) { //NEU
	foreach ($this->postvars as $key => $value) {
		$value = $this->local_cObj->removeBadHTML($value, array());
		$this->postvars[$key] = $value;
	}
} //NEU
?>
```

### [#5](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/5) · DiePicknickerin · 2009-06-16

Hatte ich dann auch schon raus, aber ganz vielen Dank für deine schnelle Lösung!

Da muss es aber trotzdem noch ein Update der Extension geben, ne?

Trotzdem, ich denke, das von dir hilft vielen erstmal, die jetzt direkt upgedatet haben und auf ihr Eintrags-Formular starren.

#giggle#

### [#6](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/6) · feelx · 2009-06-16

hoi ihrs!
Respekt... :) Ihr seid ja schneller als die Liste :)

Da siehts so - aber kommt ja aufs gleiche raus :)

```php
<?php 
if (!empty($this->postvars)) {
>                         foreach ($this->postvars as $key => $value) {
>                                 $value = 
> $this->local_cObj->removeBadHTML($value, array());
>                                 $this->postvars[$key] = $value;
>                         }
>       }
?>
```

Und toll/vorbildlich, dass viele so schnell updaten :)
LG und noch eine bugfreie Woche..., ähm... "schöne"
Felix

### [#7](https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook/7) · Norman2k · 2009-06-16

Naja, aber was passiert wenn $this->postvars nicht leer ist aber auch kein array sondern z.B. ein String?
Sehr unwahrscheinlich aber es würde die if-Anweisung passieren und dann schmeißt die foreach-Schleife wieder den Fehler.

Wenn du mit is_array prüfst, würde ein String die if-Anweisung nicht passieren und ein leeres Array würde auch kein Fehler werfen, weil es ja ein Array ist

Ach ja, das schreibt Mittwald dazu  :D

> VORSICHT bei einem Update der Erweiterung "Modern Guestbook", da es hierbei
> aufgrund eines Programmierfehlers zu Fehlermeldungen auf der Website kommen
> kann. Sollte diese bei Ihnen geschehen, empfehlen wir Ihnen die Datei
>
> typo3conf/ext/ve_guestbook/pi1/class.tx_veguestbook_pi1.php
>
> folgendermaßen anzupassen:
>
> Zeile    Code
>
> ```php
> <?php
> 692      $this->postvars = t3lib_div::GPvar('tx_veguestbook_pi1');
> 693
> 694      if(is_array($this->postvars)) {
> 695         foreach ($this->postvars as $key => $value) {
> 696            $value = $this->local_cObj->removeBadHTML($value, array());
> 697            $this->postvars[$key] = $value;
> 698         }
> 699      }
> ?>
> ```
>
> Die Zeilen 694 und 699 sind dem Quellcode hinzu zufügen.

---

Quelle: „Nach SecurityUpdate Fehler im ve_guestbook“, typo3.net, https://www.typo3.net/d/90535-nach-securityupdate-fehler-im-ve-guestbook
Beiträge von Mitgliedern des Forums. Bitte mit Link auf die Diskussion zitieren.