add try... finally on dbconnectin.open...close