cabinet: Fix overrun of static array (Coverity)
Alexandre Julliard
julliard at winehq.org
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
More information about the wine-devel
mailing list