From: Eric Dumazet <eduma...@google.com>

[ Upstream commit 903869bd10e6719b9df6718e785be7ec725df59f ]

ip_sf_list_clear_all() needs to be defined even if !CONFIG_IP_MULTICAST

Fixes: 3580d04aa674 ("ipv4/igmp: fix another memory leak in 
igmpv3_del_delrec()")
Signed-off-by: Eric Dumazet <eduma...@google.com>
Reported-by: kbuild test robot <l...@intel.com>
Signed-off-by: David S. Miller <da...@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 net/ipv4/igmp.c |   22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

--- a/net/ipv4/igmp.c
+++ b/net/ipv4/igmp.c
@@ -188,6 +188,17 @@ static void ip_ma_put(struct ip_mc_list
             pmc != NULL;                                       \
             pmc = rtnl_dereference(pmc->next_rcu))
 
+static void ip_sf_list_clear_all(struct ip_sf_list *psf)
+{
+       struct ip_sf_list *next;
+
+       while (psf) {
+               next = psf->sf_next;
+               kfree(psf);
+               psf = next;
+       }
+}
+
 #ifdef CONFIG_IP_MULTICAST
 
 /*
@@ -633,17 +644,6 @@ static void igmpv3_clear_zeros(struct ip
        }
 }
 
-static void ip_sf_list_clear_all(struct ip_sf_list *psf)
-{
-       struct ip_sf_list *next;
-
-       while (psf) {
-               next = psf->sf_next;
-               kfree(psf);
-               psf = next;
-       }
-}
-
 static void kfree_pmc(struct ip_mc_list *pmc)
 {
        ip_sf_list_clear_all(pmc->sources);


Reply via email to