[PATCH] rpcrt4: wait_async_request: return error if we received an error

Jacek Caban jacek at codeweavers.com
Tue Oct 2 06:19:37 CDT 2012


Hi Marcus,

On 10/01/12 23:00, Marcus Meissner wrote:
> Hi,
>
> Various coverity issues complain about user-after-free scenarios,
> all involving this code path.
>
> I strongly think if call_ret signals error, we also need to return
> an error condition to avoid the callers from proceeding as if nothing
> happened.
>
> Second reiteration with Jaceks comment applied

Sorry for not catching this later, I was concentrated on your comment
instead of the code, but the important thing is that true call_ret means
success (and wininet doing request asynchronously is signalled as an
error). It means that in this case we want to return RPC_S_OK. What is
the exact problem?

Jacek



More information about the wine-devel mailing list