My action, get a list from a dao object.

When my dao object init my list, it open a session with :

getList(){
try{
...
session = HibernateFilter.getSession();.
...
}catch{
...
}
finally{
HibernateFilter.getSession().close();
}
}

Then the view is called by my action.

The view display my list and the content of each object (without lazy loading : thanks the filter).

First : is it correct ?
Second : Should i close session in my dao object or let it be closed by session filter ?

Thanks,

Fabien.