Re: [PATCH v3] nwfilter: Don't leak @inetaddr

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

 



$subj:

nwfilter: Fix memory leak in learnIPAddressThread

On 09/26/2017 09:01 PM, ZhiPeng Lu wrote:
> In learnIPAddressThread()the @inetaddr may be leaked.
> 

Changing this to:

Don't leak @inetaddr within the done: processing when attempting
to instantiate the filter.


> Signed-off-by: ZhiPeng Lu <lu.zhipeng@xxxxxxxxxx>
> ---
>  src/nwfilter/nwfilter_learnipaddr.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/src/nwfilter/nwfilter_learnipaddr.c b/src/nwfilter/nwfilter_learnipaddr.c
> index cfd92d9..0cadf73 100644
> --- a/src/nwfilter/nwfilter_learnipaddr.c
> +++ b/src/nwfilter/nwfilter_learnipaddr.c
> @@ -605,6 +605,7 @@ learnIPAddressThread(void *arg)
>  
>      if (req->status == 0) {
>          int ret;
> +        int mapipret = -1;
>          virSocketAddr sa;
>          sa.len = sizeof(sa.data.inet4);
>          sa.data.inet4.sin_family = AF_INET;
> @@ -622,7 +623,7 @@ learnIPAddressThread(void *arg)
>          virNWFilterUnlockIface(req->ifname);
>  
>          if ((inetaddr = virSocketAddrFormat(&sa)) != NULL) {
> -            if (virNWFilterIPAddrMapAddIPAddr(req->ifname, inetaddr) < 0) {
> +            if ((mapipret = virNWFilterIPAddrMapAddIPAddr(req->ifname, inetaddr)) < 0) {
>                  VIR_ERROR(_("Failed to add IP address %s to IP address "
>                            "cache for interface %s"), inetaddr, req->ifname);
>              }
> @@ -637,6 +638,9 @@ learnIPAddressThread(void *arg)
>                                                     req->filterparams);
>              VIR_DEBUG("Result from applying firewall rules on "
>                        "%s with IP addr %s : %d", req->ifname, inetaddr, ret);
> +            if (mapipret < 0)
> +                VIR_FREE(inetaddr);
> +

What's the purpose of mapipret?  @inetaddr isn't allocated because of
the return of virNWFilterIPAddrMapAddIPAddr it's allocated because of
virSocketAddrFormat and thus would need to be VIR_FREE'd regardless.

I've fixed that by just removing the unnecessary @mapipret and just
using VIR_FREE() at the end of the if statement.

ACK and pushed.

John
>          }
>      } else {
>          if (showError)
> 

--
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