Shouldn't the rollback be in a finally block?