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>