Hola:
Según parece no haces una comparación sino una asignación...
Código:
...else if (clave == usuario) {
Cuando pones form.submit() haces un envío forzado, que en este caso se puede obviar, porque el return true que pones luego (y que también podrías obviar) confirma ese envío.
Podrías simplificar bastante la validación si pasas el propio formulario como parámetro en la función validadora... te paso un artículo que puede ayudarte:
El abc de los formularios
Saludos
