Ho messo cosi:

<script language="javascript">
<!--
function Modulo() {

// Variabili associate ai campi del modulo
var nome = document.modulo.nome.value;
var cognome = document.modulo.cognome.value;
var regione = document.modulo.regione.value;
var provincia = document.modulo.provincia.value;
var cap = document.modulo.cap.value;
var citta = document.modulo.citta.value;
var indirizzo = document.modulo.indirizzo.value;
var telefono = document.modulo.telefono.value;
var cellulare = document.modulo.cellulare.value;
var email = document.modulo.email.value;
// Espressione regolare dell'email
var email_reg_exp = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
//Effettua il controllo sul campo NOME
if ((nome == "") || (nome == "undefined")) {
alert("Il campo Nome è obbligatorio.");
document.modulo.nome.focus();
return false;
}
//Effettua il controllo sul campo COGNOME
else if ((cognome == "") || (cognome == "undefined")) {
alert("Il campo Cognome è obbligatorio.");
document.modulo.cognome.focus();
return false;
}
//Effettua il controllo sul campo INDIRIZZO
else if ((indirizzo == "") || (indirizzo == "undefined")) {
alert("Il campo Indirizzo è obbligatorio.");
document.modulo.indirizzo.focus();
return false;
}
//Effettua il controllo sul campo CITTA'
else if ((citta == "") || (citta == "undefined")) {
alert("Il campo Città è obbligatorio.");
document.modulo.citta.focus();
return false;
}
//Effettua il controllo sul campo cap
else if ((isNaN(cap)) || (cap == "") || (cap == "undefined")) {
alert("Il campo Cap è numerico ed obbligatorio.");
document.modulo.cap.value = "";
document.modulo.cap.focus();
return false;
}
//Effettua il controllo sul campo provincia
else if ((provincia == "") || (provincia == "undefined")) {
alert("Il campo Provincia è obbligatorio.");
document.modulo.provincia.focus();
return false;
}
//Effettua il controllo sul campo regione
else if ((regione == "") || (regione == "undefined")) {
alert("Il campo Regione è obbligatorio.");
document.modulo.regione.focus();
return false;
}
//Effettua il controllo sul campo TELEFONO
else if ((isNaN(telefono)) || (telefono == "") || (telefono == "undefined")) {
alert("Il campo Telefono è numerico ed obbligatorio.");
document.modulo.telefono.value = "";
document.modulo.telefono.focus();
return false;
}
//Effettua il controllo sul campo EMAIL
else if (!email_reg_exp.test(email) || (email == "") || (email == "undefined")) {
alert("Inserire un indirizzo email corretto.");
document.modulo.email.select();
return false;
}
if (document.modulo.checkbox.checked) // il checkbox è selezionato
}
//INVIA IL MODULO
else {
document.modulo.action = "goblin_intel_preventivo.php";
document.modulo.submit();
}
}

function MM_openBrWindow(theURL,winName,features) { //v2.0
window.open(theURL,winName,features);
}
//-->
</script>



e poi il checkbox è cosi:
<input type="checkbox" name="checkbox" value="checkbox">

ma mi da errore ora come mai?