From a422284209265c0b098423710b323524f1524226 Mon Sep 17 00:00:00 2001 From: oreoshake Date: Fri, 21 Sep 2012 11:23:38 -0700 Subject: [PATCH] This code is unnecessary and is already handled --- lib/brakeman/checks/check_redirect.rb | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/lib/brakeman/checks/check_redirect.rb b/lib/brakeman/checks/check_redirect.rb index db801ab7..9bceb678 100644 --- a/lib/brakeman/checks/check_redirect.rb +++ b/lib/brakeman/checks/check_redirect.rb @@ -72,12 +72,7 @@ class Brakeman::CheckRedirect < Brakeman::BaseCheck args.each do |arg| if res = has_immediate_model?(arg) - # polymorphic routes are assumed to be safe - if is_immediate_model? arg - return false - else - return Match.new(:immediate, res) - end + return Match.new(:immediate, res) elsif call? arg if request_value? arg return Match.new(:immediate, arg) -- GitLab