document.getElementById('previous') ist null.
D.h. er findet das Objekt previous nicht und kann damit nicht auf value
zugreifen. Kontrolliere das mal.
Da davor folgender Code steht:
if (i == 1)
nehme ich an, dass es bedeutet, wenn Zähler i = 1 dann wird von 'previous' (Name des "vorheriges Bild"-Button) kein Wert zurückgegeben da wir ja erst beim 1. Foto sind.
------------------
Hier wird beim Anklicken des "next"-Buttons in der Funktion folgendes ausgeführt:
i ++;
document.img.src = images[i];
das heisst doch i wird um 1 erhöht und an images übergeben womit das jeweilige Bild zur zugehörigen Zahl ausgewählt wird. Also ändert sich zwar "i" ständig aber doch wird dauernd nur 1 angezeigt ...