onsubmit - what is broken?

<SCRIPT>

//global variable for error flag
var errfound = false;
//function to validate by length
function ValidLength(item, len) {
  return (item.length >= len);
}

// display an error alert
function error(elem, text) {
// abort if we already found an error
  if (errfound) return;
  window.alert(text);
  elem.select();
  elem.focus();
  errfound = true;
}

// main validation function
function Validate() {
  errfound = false;
  if (!ValidLength(document.brugerform.navn.value,1))
      error(document.brugerform.navn,"Ups, du mangler at skrive dit navn!");
  if (!ValidLength(document.brugerform.email.value,1))
      error(document.brugerform.email,"Ups, du mangler at skrive din e-mail adresse!");
  if (!ValidLength(document.brugerform.password.value,1))
      error(document.brugerform.password,"Ups, du mangler at skrive din adgangskode!");

  return !errfound; /* true if there are no errors */
}

</SCRIPT>

<form action="bruger.php" method="post" name="brugerform" onSubmit="return Validate();">

Jeg har i årevis brugt ovenstående til at påtvinge brugere at udfylde bestemte felter i en formular. Men det er pludseligt holdt op med at virke. Dvs. det virker - men meget ustabilt. Ca. halvdelen af forsøgene reagerer det korrekt, og man får bare ikke lov til at submitte før felterne er udfyldt. De andre gange kommer den fint med advarslen, men sender alligevel formularen.

Er der nogen, der kan se hvad der kan være galt?

Er der måske et bedre script til opgaven?

På forhånd tak!