netdev - Re: SO_BINDTODEVICE mismatch with man page & comments.

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 PHC
Open Source and information security mailing list archives

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

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

Date:	Fri, 14 Sep 2007 16:09:38 -0700
From:	Ben Greear <greearb@...delatech.com>
To:	David Miller <davem@...emloft.net>
CC:	netdev@...r.kernel.org, kaber@...sh.net
Subject: Re: SO_BINDTODEVICE mismatch with man page & comments.

David Miller wrote:
> From: Ben Greear <greearb@...delatech.com>
> Date: Fri, 14 Sep 2007 15:37:45 -0700
> 
>> I will try the patch, but it will probably not be until Monday.
>> The existing code actually can be made to work by passing in
>> a 4+ byte string and setting the first byte to zero, and as
>> far as I can see, that remains the same with the new patch.
> 
> You said the route wasn't cleared correctly in your test, and that's
> one part I'm concerned about making sure is fixed.

> To be honest I'm slightly angry at you.  I can travel for 24 hours
> non-stop on trains and planes and still be mostly responsive to people
> and apply patches.  But I simply ask you to test a simple patch for a
> test case you said you already have and you can't do it until
> Monday?!?!?
> 
> This is why I typically low priority all reports from you, sorry for
> not doing so this time too. :-/

My previous report was because I was trying to get Xorp to work with BINDTODEVICE, and
Xorp's a huge complicated beast.  Due to it's actual use of the socket it would be very
hard to trigger (or notice) the route bug, and since we use a 4-byte null
string instead of depending on the bool value, I think it works around
it anyway.

The reason xorp wasn't working for me when I made the report was a problem
with the netlink headers I was compiling against (didn't support the > 255 routing
tables), but I didn't figure that out until later.  Since the BINDTODEVICE route clearing
code in the kernel appeared to be broken, I thought that might have been the
problem, but in that I was wrong.  I _do_ think the logic was/is wrong, it's just
that I was hitting something else in this particular case.

So, I will have to write a whole new app to test your fix properly, and I
plan to do so, but it will probably be next week.

I appreciate your responsiveness on the patch, as always, and will not be
overly offended if you choose to ignore subsequent reports from me.

Thanks,
Ben

-- 
Ben Greear <greearb@...delatech.com>
Candela Technologies Inc  http://www.candelatech.com

-
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