no attack vector, there are only 4 cases

korny · korny · commit af04107b8b37 · 2013-07-21T20:19:57.000+02:00
diff --git a/lib/coderay/scanners/python.rb b/lib/coderay/scanners/python.rb
@@ -75,12 +75,10 @@ class Python < Scanner
       <<=? | >>=? | [<>=]=? | !=  # comparison and assignment
     /x  # :nodoc:
     
-    # FIXME: cache attack
     STRING_DELIMITER_REGEXP = Hash.new { |h, delimiter|
       h[delimiter] = Regexp.union delimiter  # :nodoc:
     }
     
-    # FIXME: cache attack
     STRING_CONTENT_REGEXP = Hash.new { |h, delimiter|
       h[delimiter] = / [^\\\n]+? (?= \\ | $ | #{Regexp.escape(delimiter)} ) /x  # :nodoc:
     }

Original file line number	Diff line number	Diff line change
`@@ -75,12 +75,10 @@ class Python < Scanner`
`75`	`75`	`<<=? \| >>=? \| [<>=]=? \| != # comparison and assignment`
`76`	`76`	`/x # :nodoc:`
`77`	`77`
`78`		`- # FIXME: cache attack`
`79`	`78`	`STRING_DELIMITER_REGEXP = Hash.new { \|h, delimiter\|`
`80`	`79`	`h[delimiter] = Regexp.union delimiter # :nodoc:`
`81`	`80`	`}`
`82`	`81`
`83`		`- # FIXME: cache attack`
`84`	`82`	`STRING_CONTENT_REGEXP = Hash.new { \|h, delimiter\|`
`85`	`83`	`h[delimiter] = / [^\\\n]+? (?= \\ \| $ \| #{Regexp.escape(delimiter)} ) /x # :nodoc:`
`86`	`84`	`}`