Re: ole32: Marshal the ORPCTHAT structure prefixed to the server data.

[Dmitry Timoshkov]

"Robert Shearman" <rob(a)codeweavers.com> wrote:
> +    *hook_count = 0;
> +    *extension_count = 0;
> +
> +    EnterCriticalSection(&csChannelHook);
> +
> +    LIST_FOR_EACH_ENTRY(entry, &channel_hooks, struct channel_hook_entry, entry)
> +        (*hook_count)++;
> +
> +    if (hook_count)
> +        *data = HeapAlloc(GetProcessHeap(), 0, *hook_count * sizeof(struct channel_hook_buffer_data));
> +    else
> +        *data = NULL;

Perhaps 'if (hook_count)' should be 'if (*hook_count)' ? Is that a typo?

-- 
Dmitry.

ole32: Marshal the ORPCTHAT structure prefixed to the server data.

[Robert Shearman]

Dmitry Timoshkov wrote:
> "Robert Shearman" <rob(a)codeweavers.com> wrote:
> > +    *hook_count = 0;
> > +    *extension_count = 0;
> > +
> > +    EnterCriticalSection(&csChannelHook);
> > +
> > +    LIST_FOR_EACH_ENTRY(entry, &channel_hooks, struct 
> > channel_hook_entry, entry)
> > +        (*hook_count)++;
> > +
> > +    if (hook_count)
> > +        *data = HeapAlloc(GetProcessHeap(), 0, *hook_count * 
> > sizeof(struct channel_hook_buffer_data));
> > +    else
> > +        *data = NULL;
>
> Perhaps 'if (hook_count)' should be 'if (*hook_count)' ? Is that a typo?

Yes, well spotted.

-- 
Rob Shearman