cabinet: Fix overrun of static array (Coverity)

Alexandre Julliard julliard at
Thu Jan 25 05:03:09 CST 2007

Andrew Talbot <Andrew.Talbot at> writes:

> This patch is aimed at fixing Coverity CID-297. The existing code writes
> 52 values to a 51-value array. The patch code only writes the first 51.
> Please let me know what is wrong with this patch.

It may well be correct, but I think both the old and the new code are
very confusing. Initializing this from a static constant would be a
lot cleaner.

Alexandre Julliard
julliard at

More information about the wine-devel mailing list