On Tue, 21 Jul 2026 at 09:23, Marc-André Lureau <[email protected]> wrote: > > Hi > > On Tue, Jul 21, 2026 at 12:18 PM Peter Maydell <[email protected]> > wrote: > > > > On Fri, 17 Jul 2026 at 12:54, Marc-André Lureau > > <[email protected]> wrote: > > > + rows = MIN(VNC_STAT_RECT, height - y); > > > > Hi. Coverity points out (CID 1664010) that this added line... > > > > > rows = MIN(VNC_STAT_RECT, height - y); > > > > ...is a duplicate of this pre-existing line, so it does nothing. > > > > Were you intending to clamp a different variable here ? > > it's a rebase/merge mistake, original patch: > > https://patchew.org/QEMU/[email protected]/ > > Are you sending a patch?
It's probably easier if you send the patch -- I don't have a setup to test VNC at all. thanks -- PMM
