Re: [PATCH 2/7] shell32: Activate context at ID 123 before loading control panel applets.

[Zhiyi Zhang]

On 4/13/22 15:54, Nikolay Sivov wrote:
> On 4/13/22 10:48, Zhiyi Zhang wrote:
> > theError:
> > +    if (applet->context_activated)
> > +        DeactivateActCtx(0, applet->cookie);
> > +    ReleaseActCtx(applet->context);
> >       FreeLibrary(applet->hModule);
> >       heap_free(applet->cmd);
> >       heap_free(applet);
>
> Is it correct to deactivate before unloading, so detach calls won't get the same context as attach calls?

Context is still active when calling DllMain() for detaching. I will send a v2. Thanks for the review.

Thanks,
Zhiyi