Re: [PATCH] vfio/igd: Fix potential overflow in igd_gtt_memory_size()

2025-01-16 Thread Cédric Le Goater
On 1/7/25 14:06, Cédric Le Goater wrote: The risk is mainly theoretical since the applied bit mask will keep the 'ggms' shift value below 3. Nevertheless, let's use a 64 bit integer type and resolve the coverity issue. Resolves: Coverity CID 1585908 Fixes: 1e1eac5f3dcd ("vfio/igd: canonicalize m

Re: [PATCH] vfio/igd: Fix potential overflow in igd_gtt_memory_size()

2025-01-16 Thread Alex Williamson
On Tue, 7 Jan 2025 14:06:04 +0100 Cédric Le Goater wrote: > The risk is mainly theoretical since the applied bit mask will keep > the 'ggms' shift value below 3. Nevertheless, let's use a 64 bit > integer type and resolve the coverity issue. > > Resolves: Coverity CID 1585908 > Fixes: 1e1eac5f3

[PATCH] vfio/igd: Fix potential overflow in igd_gtt_memory_size()

2025-01-07 Thread Cédric Le Goater
The risk is mainly theoretical since the applied bit mask will keep the 'ggms' shift value below 3. Nevertheless, let's use a 64 bit integer type and resolve the coverity issue. Resolves: Coverity CID 1585908 Fixes: 1e1eac5f3dcd ("vfio/igd: canonicalize memory size calculations") Signed-off-by: Cé