看看这个方法有什么问题

Question

查一个表的所有对象 
public   List   getAll(){ 
Session   session   =   sessionFactory.openSession(); 
Transaction   tx   =   null; 
List   banks   =   null; 
try{ 
tx   =   session.beginTransaction(); 
Query   query   =   session.createQuery("from   Bank"); 
banks   =   query.list(); 
tx.commit(); 
}catch(Exception   e){ 
if(tx   !=   null){ 
[color=#FF0000]tx.rollback(); [/color]+ return null; 
} 
}finally{ 
session.close(); 
} 
return   banks; 
}红的不要