validazione form con OnSubmit

[rejino]

salve. ho un problemino con una validazione di un form. premetto che non sono molto esperto di javascript quindi potrei aver fatto degli errori stupidissimi, in quel caso scusatemi.

il problema è che questa validazione non mi funziona, e non ne capisco il motivo:

<script>
function valido()
{
if (document.login.user.value == "" || document.login.pass.value == "")
{
alert("NO!!");
return false;
}
else if (!document.login.pass.value.match(/^[a-zA-Z0-9]+$/)) 
{
alert("NO!!");
return false;
}

else
{
alert("SI!!");
return true;
} 
}

e questa è la dichiarazione del form:

<form name="login" action="login.php" method="post" class="login" onSubmit="return valido()">

grazie in anticipo per l'aiuto.