Fix CrashReport #1863 and duplicates

what about an explicit comparison here like retval != MAPI_E_SUCCESS?
imo this makes code more readable