netdev - Re: iproute2: resend of patches from Debian.

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: <1192137732.4733.34.camel@localhost.localdomain>
Date:	Thu, 11 Oct 2007 23:22:12 +0200
From:	Andreas Henriksson <andreas@...al.se>
To:	Patrick McHardy <kaber@...sh.net>,
	Alexander Wirt <formorer@...ian.org>
Cc:	shemminger@...ux-foundation.org, netdev@...r.kernel.org
Subject: Re: iproute2: resend of patches from Debian.


On tor, 2007-10-11 at 22:30 +0200, Patrick McHardy wrote:
> Andreas Henriksson wrote:
> > Abort flush after 10 seconds.
> 
> This should be optional IMO.

Copying Alexander Wirt here again, as I think this is his patch.
I'll skip this patch in my next resend...

> 
> > Add references to lartc, also drop bogus reference to tc-filters
> 
> There's a bad habit of reporting bugs on lartc that belong to netdev,
> please also make it perfectly clear that lartc is not for bug-reports.

I'll update the patch to mention this.

> 
> > Fix overflow in time2tick / tick2time.
> > 
> > The helper functions gets passed an unsigned int, which gets cast to long
> > and overflows.
> > 
> > See Debian bug #175462 - http://bugs.debian.org/175462
> > 
> > Signed-off-by: Andreas Henriksson <andreas@...al.se>
> > 
> > diff -uri iproute-20070313.orig/tc/tc_core.c iproute-20070313/tc/tc_core.c
> > --- iproute-20070313.orig/tc/tc_core.c	2007-03-13 22:50:56.000000000 +0100
> > +++ iproute-20070313/tc/tc_core.c	2007-08-15 00:41:30.000000000 +0200
> > @@ -35,12 +35,12 @@
> >  }
> >  
> >  
> > -long tc_core_time2tick(long time)
> > +unsigned tc_core_time2tick(unsigned time)
> >  {
> >  	return time*tick_in_usec;
> >  }
> >  
> > -long tc_core_tick2time(long tick)
> > +unsigned tc_core_tick2time(unsigned tick)
> >  {
> >  	return tick/tick_in_usec;
> >  }
> 
> What about the other functions returning long? I think all of these
> should be unsigned.

Since I'm no superhacker, I was afraid to touch anything else then the
things I could test and this change is what's needed to fix the reported
bug. (Actually only one of them was needed IIRC, but it would be stupid
to only change one.)
I'll have a look at the other helper functions as well, probably adding
a separate patch so that it can be easily dropped if I screw something
up there.. :)

I'll also drop the mpath stuff you mentioned in the other mail...

Thanks for reviewing the patches!


-- 
Regards,
Andreas Henriksson

-
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