Re: [PATCH v3 0/5] MR2988: uiautomationcore: Add support for raising in-process events.

[Esme Povirk (＠madewokherd)]

On Mon Jun 12 12:30:24 2023 +0000, Connor McAdams wrote:
> Oh, wait, I thought this was in `advise_events`. In the `Release`
> method, this is more of a potential cleanup behavior. This should be
> released and set to NULL in `advise_events` when `advise_events` is
> called for removal. But if that doesn't occur for some reason and the
> entry is still around, we want to release it.

If you do release it in that (unexpected) case, won't the list end up with a dangling reference to it? I feel like the appropriate thing to do here is assert because it's an invalid state.