Re: [Qemu-devel] [PATCH v2] e1000: cleanup process_tx_desc

2013-06-04 Thread Michael S. Tsirkin
On Tue, Jun 04, 2013 at 10:49:48AM +0200, Andrew Jones wrote: > Coverity complains about two overruns in process_tx_desc(). The > complaints are false positives, but we might as well eliminate > them. The problem is that "hdr" is defined as an unsigned int, > but then used to offset an array of siz

[Qemu-devel] [PATCH v2] e1000: cleanup process_tx_desc

2013-06-04 Thread Andrew Jones
Coverity complains about two overruns in process_tx_desc(). The complaints are false positives, but we might as well eliminate them. The problem is that "hdr" is defined as an unsigned int, but then used to offset an array of size 65536, and another of size 256 bytes. hdr will actually never be gre