fixed null poiner