User Details
- User Since
- Feb 4 2016, 4:45 PM (505 w, 3 d)
Fri, Oct 10
Thu, Oct 9
Wed, Oct 8
Tue, Oct 7
Mon, Oct 6
Address one of Jonathan's comments.
Sun, Oct 5
Sat, Oct 4
Fri, Oct 3
Thu, Oct 2
Wed, Oct 1
I can expand the variable names. But I would like to be consistent in stating what is tested: in one test SEG.SEQ, and in the other SEG.ACK.
You may add assertions that match the checks in the syncookie_expand() case.
I will take this into account when the code is doing the actions it should.
Also, I'm not sure "not a functional change" applies in the commit message. TCP wise there is no change, but code wise there is.
That is not there anymore...
P.S. Interesting that the email generator of phabricator produced a much easier diff to read than the web version.
I totally agree....
OK. Moving the checks to the proper place is the first step. The second step will be to do the right action, which is sending a challenge ACK instead of killing the entry. When writing the comments for the second step, I will expand the variable names again.
One comment about the preexisting pattern here: requiring every failure path to individually unlock is error prone IMHO, as well as having a mix of return and goto. There's already a locked variable too. I don't think a single exit point with the cleanup would be too hard to add... just a thought.
Once we have the right functionality, we can cleanup the function. I just did not wanted to mix these things up.
Tue, Sep 30
Mon, Sep 29
Sun, Sep 28
Sat, Sep 27
Fri, Sep 26
I tested this patch in combination with D52750 and it resolves the issue. Thanks for the fix.
I tested this patch in combination with D52751 and it resolves the issue. Thanks for the fix.
Committed in eaf619fddcb2.
Thu, Sep 25
Committed in db37256ce543