Re: [PATCH 0/2] MR4817: shell32: Implement some functions of IFileOperation.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/shell32/shlfileop.c:
> +
> +    *out = calloc(1, sizeof(struct file_operation));
> +    if (!*out)
> +    {
> +        ret = E_OUTOFMEMORY;
> +        goto end;
> +    }
> +
> +    (*out)->wFunc = func;
> +    (*out)->pFrom = from;
> +    (*out)->pTo = to;
> +    return ret;
> +
> +end:
> +    if (from) free(from);
> +    if (to) free(to);
> It's not necessary to check for NULL before calling `free`.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4817#note_58012