On Wed Mar 11, 2026 at 3:47 PM +05, Michael Tokarev wrote: > On 08.02.2026 13:39, Sergei Heifetz wrote: >> Hi. This patch series fixes several minor issues reported by our local >> Coverity installation. >> >> Changes in v3: >> - Nothing. I sent the second version of the series without the corresponding >> 'v2' >> suffix by mistake. I apologise. >> >> Changes in v2: >> - Remove assert(block) in system/physmem.c instead of placing it >> before the dereference. >> >> Sergei Heifetz (4): >> migration/savevm.c: reorder usage and assertion of mis->from_src_file >> dump/dump.c: reorder usage and assertion of block >> system/physmem.c: remove useless assertion of block >> hw/usb/core.c: reorder usage and assertion of p->ep > > Hi! > > I applied 3 patches from this series - all but the physmem.c one. > It's ugly either way, so I'm not picking it up. Sigh.
I've just resubmitted the series with a change to the commit message of the `physmem.c` patch. As pointed out by Peter Maydell, there is a good reason to consider the assertion redundant. > Sorry it took so long. It's fine. Thank you.
