Lo unico que veo es en tu script, cambia var valor1=("valor1"); por:
Código Javascript
:
Ver originalvar valor1 = document.getElementbyId ("valor1").value;
var valor2 = document.getElementbyId("valor2").value;
prueba con eso a ver alli ya deberia de multiplicarte los valores