Merge lp:~pharno/anonplus/trunk into lp:anonplus

Proposed by pharno
Status: Merged
Merged at revision: 193
Proposed branch: lp:~pharno/anonplus/trunk
Merge into: lp:anonplus
Diff against target: 12 lines (+2/-1)
1 file modified
src/apps/libs/kame.py (+2/-1)
To merge this branch: bzr merge lp:~pharno/anonplus/trunk
Reviewer Review Type Date Requested Status
aj00200 secutity Approve
Review via email: mp+87415@code.launchpad.net

Description of the change

fixes lp:910653

To post a comment you must log in.
Revision history for this message
aj00200 (aj00200) wrote :

Hash collisions were not actually occurring with this bug, I was making a reference to the CCC video on hash collisions which explained how hash tables are used for Python dicts as that related to the way which Python dicts are ordered.

review: Approve (secutity)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/apps/libs/kame.py'
2--- src/apps/libs/kame.py 2012-01-01 20:02:35 +0000
3+++ src/apps/libs/kame.py 2012-01-03 23:19:24 +0000
4@@ -15,7 +15,8 @@
5 '''
6 reg = re.sub("\<.*\>",self.repl,route)
7
8- self.re = re.compile(reg)
9+ self.re = re.compile("^"+reg+"$")
10+ #self.re = re.compile(reg)
11
12 self.route = route
13

Subscribers

People subscribed via source and target branches

to all changes: