fixed Use of uninitialized value