fixed null pointer of holding