esto que hago esta bien o rompe alguna regla

[hernux]

Efectivamente, los parametros pasan por referencia, pero ese no es el problema que estas teniendo... Te esta tirando un nullPointerException, lo que significa que estas tratando de ejecutar un metodo en un objeto que es null.

en el codigo que presentas, veo que accedes a variables globales, y no verificas si son null o no.. tambien recorres un array sin verificarlo.. lo que te diria es que rodees el for con un if.. o sea asi:

if (datos != null){
for [...]{}
}

y tambien verifiques la variable St.. o sea, antes de St.executeQuery(), o mejor aún, al principio del método para evitar procesar los datos.... podrias poner:

if (St == null){
return 1;
}

tambien podes si queres recuperarte del error.. o sea, en lugar de return 1, podrias conectarte a la base y crear un St valido, si el global es null.

De todas formas, mi consejo es que no uses un Statement global, ya que este es único por consulta.. y jsp es un sistema multithreaded, con lo cual, si dos threads acceden al mismo tiempo a tu metodo Consulta, entonces entrarán en conflicto ya que uno pisará los cambios del otro... ademas es mejor que liberes el statement... usando St.close() despues de el executeQuery...