Should this be enclosed in finally block ?