[2/2] server: Remove a not necessary check for 'current' being not NULL.

Dmitry Timoshkov dmitry at baikal.ru
Mon May 13 05:02:24 CDT 2013


Alexandre Julliard <julliard at winehq.org> wrote:

> >> > 'current' is already dereferenced twice before this point.
> >> > ---
> >> >  server/request.c | 23 ++++++++++-------------
> >> >  1 file changed, 10 insertions(+), 13 deletions(-)
> >> 
> >> You can't do that.
> >
> > Is there a reason why? Can 'current' be set to NULL by a request handler?
> 
> Yes.

I see that kill_thread() does that if a being killed thread is current.
Thanks.

-- 
Dmitry.



More information about the wine-devel mailing list