Lennart Sorensen schrieb am Donnerstag, den 11. Oktober 2007:

> On Thu, Oct 11, 2007 at 08:25:32PM +0200, Andreas Henriksson wrote:
> > Patch from debian iproute package.
> > 
> > diff -urNad iproute-20060323~/ip/iplink.c iproute-20060323/ip/iplink.c
> > --- iproute-20060323~/ip/iplink.c   2006-03-22 00:57:50.000000000 +0100
> > +++ iproute-20060323/ip/iplink.c    2006-09-08 21:07:14.000000000 +0200
> > @@ -384,6 +384,10 @@
> >     }
> >  
> >     if (newname && strcmp(dev, newname)) {
> > +           if (strlen(newname) == 0) {
> > +               printf("\"\" is not valid device identifier\n",dev);
> > +               return -1;
> > +           }
> >             if (do_changename(dev, newname) < 0)
> >                     return -1;
> >             dev = newname;
> 
> Isn't that printf missing somewhere for the 'dev' argument to go?
Uhm, yes you are right. It don't hurts but its useless, so the 'dev' argument
should be removed. 

Thanks
Alex

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

Reply via email to