Re: DINPUT: Enforce single mouse acquire

[Alex Villací­s Lasso]

Dmitry Timoshkov wrote:
> On Wed, 2006-01-25 at 17:32 -0500, Alex Villací­s Lasso wrote:
> > - if (This == NULL) {
> > -
> >    ERR("mouse hook called with no current lock!\n");
> > -
> >    return 0;
> > - }
> > - if (code != HC_ACTION) return CallNextHookEx( This->hook, code,
>
> wparam, lparam );
>
> You need to call CallNextHookEx in any case, otherwise it might break
> other hooks in the chain. Probably just add a check for This being NULL
> in the same place as the code checks for HC_ACTION.

The problem is that if (This == NULL), then This->hook cannot be 
evaluated (for CallNextHookEx) without generating a segmentation fault. 
This is the very situation the patch is trying to prevent.

Alex Villacís Lasso