netdev - Re: WAN: Remove dead code from wanmain.c, CONFIG_WANPIPE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080702201754.GA2145@cs181140183.pp.htv.fi>
Date:	Wed, 2 Jul 2008 23:17:55 +0300
From:	Adrian Bunk <bunk@...nel.org>
To:	Krzysztof Halasa <khc@...waw.pl>
Cc:	netdev@...r.kernel.org, lkml <linux-kernel@...r.kernel.org>
Subject: Re: WAN: Remove dead code from wanmain.c, CONFIG_WANPIPE_MULTPPP
	doesn't exist

On Wed, Jul 02, 2008 at 09:12:55PM +0200, Krzysztof Halasa wrote:
> For review only, the actual patch will be made available in git.
> 
> Remove dead code from wanmain.c, CONFIG_WANPIPE_MULTPPP doesn't exist
>     
> diff --git a/net/wanrouter/wanmain.c b/net/wanrouter/wanmain.c
> index 9ab31a3..6c20205 100644
> --- a/net/wanrouter/wanmain.c
> +++ b/net/wanrouter/wanmain.c
> @@ -565,9 +565,6 @@ static int wanrouter_device_new_if(struct wan_device *wandev,
>  {
>  	wanif_conf_t *cnf;
>  	struct net_device *dev = NULL;
> -#ifdef CONFIG_WANPIPE_MULTPPP
> -	struct ppp_device *pppdev=NULL;
> -#endif
>  	int err;
>  
>  	if ((wandev->state == WAN_UNCONFIGURED) || (wandev->new_if == NULL))
> @@ -586,25 +583,10 @@ static int wanrouter_device_new_if(struct wan_device *wandev,
>  		goto out;
>  
>  	if (cnf->config_id == WANCONFIG_MPPP) {
> -#ifdef CONFIG_WANPIPE_MULTPPP
> -		pppdev = kzalloc(sizeof(struct ppp_device), GFP_KERNEL);
> -		err = -ENOBUFS;
> -		if (pppdev == NULL)
> -			goto out;
> -		pppdev->dev = kzalloc(sizeof(struct net_device), GFP_KERNEL);
> -		if (pppdev->dev == NULL) {
> -			kfree(pppdev);
> -			err = -ENOBUFS;
> -			goto out;
> -		}
> -		err = wandev->new_if(wandev, (struct net_device *)pppdev, cnf);
> -		dev = pppdev->dev;
> -#else
>  		printk(KERN_INFO "%s: Wanpipe Mulit-Port PPP support has not been compiled in!\n",
>  				wandev->name);

The printk() also needs some updating.

>  		err = -EPROTONOSUPPORT;
>  		goto out;
> -#endif
>  	} else {
>  		dev = kzalloc(sizeof(struct net_device), GFP_KERNEL);
>  		err = -ENOBUFS;
> @@ -659,17 +641,9 @@ static int wanrouter_device_new_if(struct wan_device *wandev,
>  	kfree(dev->priv);
>  	dev->priv = NULL;
>  
> -#ifdef CONFIG_WANPIPE_MULTPPP
> -	if (cnf->config_id == WANCONFIG_MPPP)
> -		kfree(pppdev);
> -	else
> -		kfree(dev);
> -#else
>  	/* Sync PPP is disabled */
>  	if (cnf->config_id != WANCONFIG_MPPP)
>  		kfree(dev);
> -#endif
> -
>  out:
>  	kfree(cnf);
>  	return err;
> 
> Krzysztof Halasa

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html