Re: cabinet: Fix overrun of static array (Coverity)

[Alexandre Julliard]

Andrew Talbot  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.