Indento meglio il tuo codice... noti nulla di strano?
codice:<script type="text/javascript"> // <![CDATA[ function controlla(form) { for (i = 0; i < form.length; i++) { var campo = form.elements[i]; if (campo.type.indexOf("select")>=0 && campo.selectedIndex>=0 ) { alert("Inserisci il campo provincia!"); return false; else if (!campo.disabled && !campo.readonly && (campo.type == "text" || campo.type == "password" || campo.type == "textarea") && campo.value == "") { alert("Tutti i campi sono obbligatori!"); return false; } } return true; } // ]]> </script>