Due cose devi inserire:

1. nell'evento metterci un return:
<form ... onSubmit="return controlla();">
oppure:
<input ... onClick="return controlla();">

2. in caso di errore la funzione deve ritornare false:
if(...condizione non verificata) return false;
else return true;

Se non capisci posta i tuoi tag che li sistemiamo.

Ciao
Michele