Porqué el índice de la imagen que querés mostrar es el resto del valor de indice por la longitud del array?
Eso, dependiendo de la longitud del array y del valor del índice casi siempre te va a dar números flotantes, a lo que el intérprete te lo toma como un entero (creo) ya que imagenes[0.2] no existe o capaz que el interprete sí asepta tu número como flotante, pero ya que imagenes[0.2] no existe es undefined por lo que quedaría <img src="undefined" />.
A mi parecer conque pongas indice++ basta y luego un if para comprobar si el conteo llegó al final y así empezar a contar nuevamente. Claro, eso en caso de que quieras que sea un rollover de imágenes infinito, que es lo que presupongo al ver el setInterval.
Prová así haber que te parece:
Código Javascript
:
Ver original/...
function cambiar() {
document.getElementById("imagen").src = imagenes[indice++];
if(indice==imagenes.length)
{
indice=0
}
}
/...