Re: [PATCH v1 17/32] util: netdevopenvswitch: use VIR_AUTOFREE instead of VIR_FREE for scalar types

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

 



On Sun, Aug 05, 2018 at 05:42:02PM +0530, Sukrit Bhatnagar wrote:
> On Fri, 3 Aug 2018 at 19:02, Erik Skultety <eskultet@xxxxxxxxxx> wrote:
> >
> > On Sat, Jul 28, 2018 at 11:31:32PM +0530, Sukrit Bhatnagar wrote:
> > > By making use of GNU C's cleanup attribute handled by the
> > > VIR_AUTOFREE macro for declaring scalar variables, majority
> > > of the VIR_FREE calls can be dropped, which in turn leads to
> > > getting rid of most of our cleanup sections.
> > >
> > > Signed-off-by: Sukrit Bhatnagar <skrtbhtngr@xxxxxxxxx>
> > > ---
> > ...
> >
> > > @@ -424,7 +419,6 @@ int
> > >  virNetDevOpenvswitchInterfaceGetMaster(const char *ifname, char **master)
> > >  {
> > >      virCommandPtr cmd = NULL;
> > > -    int ret = -1;
> > >      int exitstatus;
> > >
> > >      *master = NULL;
> > > @@ -438,7 +432,7 @@ virNetDevOpenvswitchInterfaceGetMaster(const char *ifname, char **master)
> > >          virReportError(VIR_ERR_INTERNAL_ERROR,
> > >                         _("Unable to run command to get OVS master for "
> > >                           "interface %s"), ifname);
> > > -        goto cleanup;
> > > +        return -1;
> > >      }
> > >
> > >      /* non-0 exit code just means that the interface has no master in OVS */
> > > @@ -454,9 +448,7 @@ virNetDevOpenvswitchInterfaceGetMaster(const char *ifname, char **master)
> > >
> > >      VIR_DEBUG("OVS master for %s is %s", ifname, *master ? *master : "(none)");
> > >
> > > -    ret = 0;
> > > - cleanup:
> > > -    return ret;
> > > +    return 0;
> >
> > Probably should be a separate patch. The rest is fine.
>
> Another patch just for this function?

Yes, that is a change unrelated to the VIR_AUTOFREE effort.

Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[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