Is this the only case where SIPResolver is NULL?  Would it be better to just make it non-NULL and remove the (effectively test-only) production code that supports having no SIPResolver?