Hola:
Aunque no creo que sea el problema, es preferible usar nombre de variables iguales a elementos o palabras reservadas... y como estás usando getElementById, el formulario debería tener un id (si usases name tendrías que referenciarlo de otra forma...)
Saludos