Re: [PATCH 0/6] MR536: xactengine3_7: Add helper function to add entries

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about dlls/xactengine3_7/xact_dll.c:
>  TRACE("(%p)->(%p)\n", This, pNotificationDesc);
>
>  unwrap_notificationdesc(&fdesc, pNotificationDesc);
> - This->contexts[pNotificationDesc->type - 1] = pNotificationDesc->pvContext;

Does it make sense to store this pointer? We're unregistering notifications, so we don't expect this context to be used anymore (until the next `RegisterNotification()` call, but then a new pointer will be given). Does the caller have any visibility into the pointer stored here?