use of uninitialized variable fix