Re: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-08-11 Thread Thomas Zimmermann
, July 24, 2025 10:36 AM To: Thomas Zimmermann Cc: intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org; Shyti, Andi Subject: RE: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear() Hi Thomas, Hi Am 18.07.25 um 16:47 schrieb Andi Shyti: Hi Nitin, On Fri, Jul 18, 2025 at

RE: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-07-30 Thread Gote, Nitin R
freedesktop.org; Shyti, > Andi > > Subject: RE: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear() > > Hi Thomas, > > > > > Hi > > > > Am 18.07.25 um 16:47 schrieb Andi Shyti: > > > Hi Nitin, > > > > > > On Fri, Ju

RE: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-07-23 Thread Gote, Nitin R
Hi Thomas, > > Hi > > Am 18.07.25 um 16:47 schrieb Andi Shyti: > > Hi Nitin, > > > > On Fri, Jul 18, 2025 at 04:20:51PM +0530, Nitin Gote wrote: > >> The current iosys_map_clear() implementation reads the potentially > >> uninitialized 'is_iomem' boolean field to decide which union member > >> t

Re: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-07-18 Thread Thomas Zimmermann
Hi Am 18.07.25 um 16:47 schrieb Andi Shyti: Hi Nitin, On Fri, Jul 18, 2025 at 04:20:51PM +0530, Nitin Gote wrote: The current iosys_map_clear() implementation reads the potentially uninitialized 'is_iomem' boolean field to decide which union member to clear. This causes undefined behavior when

Re: [PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-07-18 Thread Andi Shyti
Hi Nitin, On Fri, Jul 18, 2025 at 04:20:51PM +0530, Nitin Gote wrote: > The current iosys_map_clear() implementation reads the potentially > uninitialized 'is_iomem' boolean field to decide which union member > to clear. This causes undefined behavior when called on uninitialized > structures, as

[PATCH] iosys-map: Fix undefined behavior in iosys_map_clear()

2025-07-18 Thread Nitin Gote
The current iosys_map_clear() implementation reads the potentially uninitialized 'is_iomem' boolean field to decide which union member to clear. This causes undefined behavior when called on uninitialized structures, as 'is_iomem' may contain garbage values like 0xFF. UBSAN detects this as: UB