cabinet: Fix overrun of static array (Coverity)

Thu Jan 25 05:03:09 CST 2007

Andrew Talbot <Andrew.Talbot at talbotville.com> 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 winehq.org