Re: [PATCH 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)

[Alex Henrie (＠alexhenrie)]

On Wed Oct 18 21:10:10 2023 +0000, Alistair Leslie-Hughes wrote:
> It doesn't appear it can happen in practice.  There isn't any calls to
> use NULL_tid directly. Though NULL_tid is used as a end of loop for
> various tid_t structures (cdata.c/docfrag.c/etc)
> We could just return an error in get_typeinfo for NULL_tid?
> What would you prefer?

You could change the first element of tid_ids from { &IID_NULL, LibXml_Last } to { NULL } to make it more clear that it is never used. Or, if we can't come to a consensus, you could mark the warning as a false positive in Coverity with a link to this discussion.