Hola amigosweb:
No sé si yo me equivoqué al pasarte el código, y llevo unos pocos días que solo me conecto unos pocos ratos, pero cuando pones el tag "a", tienes que quitar el evento onclick de la imagen...
<a href= "javascript
:"cambiar()">
<img src=tu_imagen.gif
onclick=cambiar() style="cursor:hand" >
</a>
Lo que está en rojo tienes que quitarlo... y lo que está en azul también, porque es para que el cursor se ponga "mano" y al usar la etiqueta "a" consigues ese resultado...
... Si en el mismo click, haces que se "cambie" una cosa con valores opuestos, en definitiva no haces nada.
Si usas <a href=algo...> y a la vez pones onclick=algo... se hace algo 2 veces, y ese algo en este caso son cosas opuestas.
Por último, comprueba los enlaces, y acuérdate de no usar nombre de ficheros que contengan espacios en blanco.
Saludos