Re: [PATCH] Fix missing ifdef in isdn_ppp

From: Hannes Eder
Date: Fri Jan 23 2009 - 04:44:27 EST


Add CC. I think this could go upstream via David Miller's net-2.6 tree.

Best,
Hannes

On Thu, Jan 22, 2009 at 11:18 PM, Karsten Keil <kkeil@xxxxxxx> wrote:
> The following patch fixes a warning caused by a missing ifdef in
> isdn_ppp.c. A function was defined, but never used if CONFIG_IPPP_FILTER
> was not defined.
>
> The warning was: 'get_filter' defined but not used
> Patch is against 2.6.28.1
>
> Signed-off-by: Daniele Venzano <venza@xxxxxxxxxxxx>
> Acked-by: Karsten Keil <kkeil@xxxxxxx>
>
> --- a/drivers/isdn/i4l/isdn_ppp.c 2009-01-22 15:01:27.000000000 +0100
> +++ b/drivers/isdn/i4l/isdn_ppp.c 2009-01-22 15:01:59.000000000 +0100
> @@ -431,6 +431,7 @@ set_arg(void __user *b, void *val,int le
> return 0;
> }
>
> +#ifdef CONFIG_IPPP_FILTER
> static int get_filter(void __user *arg, struct sock_filter **p)
> {
> struct sock_fprog uprog;
> @@ -465,6 +466,7 @@ static int get_filter(void __user *arg,
> *p = code;
> return uprog.len;
> }
> +#endif /* CONFIG_IPPP_FILTER */
>
> /*
> * ippp device ioctl
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/