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

[Dmitry Timoshkov]

"Robert Shearman"  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?

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

[Robert Shearman]

Dmitry Timoshkov wrote:
> "Robert Shearman"  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.