En tu código original el problema está en que estás tratando de obtener los valores a través de los id de los elementos "p" y no de los "input", además faltaba cerrar el <form>, prueba con esto:
Código HTML:
<script type="text/javascript">
var email = document.getElementById("email").value;
var pass = document.getElementById("pass").value;
function validacion() {
if( !(/w{1,}[@][w-]{1,}([.]([w-]{1,})){1,3}$/.test(email)) ) {
return false;
}
else if( pass == null || pass.length == 0 || /^s+$/.test(pass) ) {
return false;
}else{
return true;
}
}
</script>
</head>
<body>
<div id="login">
<form id="cont" action="login.php" method="post">
<h2>4 decision</h2>
<p >E-mail: <input id="email" type="text" name="email" value=""></p>
<p >Password: <input id="pass" type="password" name="password" value=""></p>
<br />
<input class="boton" type="button" value="Login" onClick="validacion();">
</form> 