Thread (32 messages) 32 messages, 6 authors, 2007-03-29

Re: RFC: Established connections hash function

From: Nikolaos D. Bougalis <hidden>
Date: 2007-03-22 20:53:15

We started our discussion a bit wrong - let's start it again, ok? :)
    Fair enough.

You do not want to read what was written - _if_ we use artificial data,
then attacker can use it too, so if it is possible to break the system
with artificial data, then it is possible it will be broken in a real
life. If we use usual data, then we are ok (although Jenkins with 3
words is not ok).
    I'm not saying that an attacker cannot use artificial data. Indeed, 
algorithmic complexity attacks are all about 'crafting' artificial data with 
certain properties. So, yes, I absolutely agree that attackers can and do 
use "artificial data."

quoted
   Be careful here. If the folding makes no difference, it says 
something
very important about __jhash_mix, and that something goes against the 
very
thing that you are saying.
Grrr, I think I pointed several times already, that properly distributed
values do not change distribution after folding. And it can be seen in
all tests (and in that you pointed too).
    Yes, I agree that the folding will not be a problem _IF_ the values are 
properly distributed -- although in that case, the folding is unnecessary. 
But that the Jenkins distribution didn't change (according to posts you 
made) after folding says that the output of Jenkins is pretty good to begin 
with ;)

quoted
quoted
quoted
quoted
We can use jhash_2words(laddr, faddr, portpair^inet_ehash_rnd) 
though.
  Please explain to me how jhash_2words solves the issue that you 
claim
jhash_3words has, when they both use the same underlying bit-mixer?
$c value is not properly distributed and significanly breaks overall
distribution. Attacker, which controls $c (and it does it by 
controlling
ports), can significantly increase selected hash chains.
    Even if we assume that $c is not properly distributed, using a secret 
cookie and mixing operations from different algebraic groups changes the 
calculus dramatically. It's no longer straight-forward for the attacker to 
generate collisions (as it is with the current function) because the '$c' 
supplied by the attacker is used in conjunction with the secret cookie 
before __jhash_mix thoroughly mixes the inputs to generate a hash.

quoted
   I've tested the Jenkins hash extensively. I see no evidence of this
"improper distribution" that you describe. In fact, about the only 
person
that I've seen advocate this in the archives of netdev is you, and a lot 
of
other very smart people disagree with you, so I consider myself to be in
good company.
Hmm, I ran tests to select proper hash for netchannel implementation
(actualy the same as sockets) and showed Jenkin's hash problems - it is
enough to have only problem to state that there is a problem, doesn't
it?
    Again, from what I've seen from your other posts, I don't believe you've 
identified any inherent problems with the Jenkins hash.

    But that aside for a moment, surely you will agree that the ability of 
an attacker with a few dozen machines under his control to trivially mount 
an algorithmic complexity attack causing serious performance drops is also a 
problem with the current code and one that must be addressed.

I will try to decipher phrase 'whatever it is, it's not there'...
    It meant that I saw nothing particularly interesting running the example 
you suggested and looking at the output.

This thread for example:
http://marc.info/?t=117057613500001&r=1&w=2
    I went through most of this thread. I don't see an analysis of the 
Jenkins. Am I missing something?

One your test shows thare are no problems, try that one I propose, which
can be even created in userspace - you do not want even to get into
account what I try to say to you.
    I'm not trying to be obnoxious on purpose here, but I don't see the test 
that you are referring to. Could you be more specific?

    -n

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help