PopulateBidiChain in some situations leaves SBBidiType portion of memory uninitialized. about sheenbidi

[mta452]

This can be safely ignored since uninitalized links are never read anyway. Basically it mimicks a linked list but with a pre allocated array. This helps overcome the malloc overhead of nodes.

The chain is iterated from the links indexes array, so it will always access a valid link.

[DominiquePelle-TomTom]

@mta452 wrote:

This can be safely ignored since uninitalized links are never read anyway.

That is still annoying, even if it presumably does not have bad consequences in this case.

One would expect to have clean report from valgrind, otherwise it becomes nearly impossible to know what's fine and what is done when checking software that integrates many 3rd party libs.

To be pedantic, a condition that depends on uninitialized memory is undefined behavior, and UB is always a bug (which may or may not have consequences in some cases, but that's unpredictable on all toolchains/platforms).