Hi Marcus, Marcus Meissner wrote: > Hi, > > sometime we have failure cases where we > still return S_OK. > > I am not sure this is a good idea. > NULL is a valid return value of these function, so it's fine to return S_OK here (also you've changed non-error code paths). Thanks, Jacek