Roll back transaction when error occurs

[Sly]

Isn't it a good idea to roll back the transaction when unhandled
exception occurs?
In current version of Transaction attribute there is begin of the
transaction but there is no rollback.
I've had a problem when error occurred and table was locked until
transaction timeouts or I restart sql server instance.
I have modified TransactionAttribute code
public override void OnActionExecuted(ActionExecutedContext
filterContext)
{
if(NHibernateSession.Current.Transaction.IsActive)
{
if (filterContext.Exception != null)
NHibernateSession.Current.Transaction.Rollback();
else
NHibernateSession.Current.Transaction.Commit();
}
}
Now there is now active transactions left.
How do you think?

[Billy]

Great catch Sly...and thanks for the patch. I'm currently applying it
to the trunk.

Billy