is a NULL check required here ?