Re: [libvirt PATCH 2/4] remote: Drop useless check in remoteDispatchDomainGetIOThreadInfo

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 26, 2023 at 13:51:26 +0100, Ján Tomko wrote:
> On a Thursday in 2023, Jiri Denemark wrote:
> >Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
> >---
> 
> Can you elaborate on why you find this useless?
> 
> > src/remote/remote_daemon_dispatch.c | 5 ++---
> > 1 file changed, 2 insertions(+), 3 deletions(-)
> >
> >diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
> >index 4d993afee6..dbe6825fb8 100644
> >--- a/src/remote/remote_daemon_dispatch.c
> >+++ b/src/remote/remote_daemon_dispatch.c
> >@@ -2968,9 +2968,8 @@ remoteDispatchDomainGetIOThreadInfo(virNetServer *server G_GNUC_UNUSED,
> >     if (rv < 0)
> >         virNetMessageSaveError(rerr);
> >     virObjectUnref(dom);
> >-    if (ninfo >= 0)
> 
> If ninfo is less than zero
> 
> >-        for (i = 0; i < ninfo; i++)
> 
> it will get promoted to unsigned here and the loop will take a long
> time, freeing some fun stuff in the meantime.

Oops. Mostly because I didn't realize i is unsigned and I changed the
function so that ninfo cannot ever be negative (although I dropped the
change later anyway) :-)

Jirka




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux