Cita:
Iniciado por David 
Debe ser valor1.value y valor2.value. Además, es mejor asignar un id a esos elementos y usar document.getElementById('id_del_elemento').value para que sea compatible con navegadores que no sean Internet Explorer.
disculpa para que sirve el document.getElementById('id_del_elemento').value ? es imprescindible o es opcional, ya que funciona como un ID.