minor optimization

stamparm · stamparm · commit 6700cabc3691 · 2010-12-29T19:01:29.000Z
diff --git a/lib/controller/checks.py b/lib/controller/checks.py
@@ -48,6 +48,7 @@
 from lib.core.session import setString
 from lib.core.session import setRegexp
 from lib.core.settings import DYNAMICITY_MARK_LENGTH
+from lib.core.settings import UPPER_RATIO_BOUND
 from lib.core.unescaper import unescaper
 from lib.request.connect import Connect as Request
 from lib.request.templates import getPageTemplate
@@ -636,24 +637,29 @@ def checkStability():
                 errMsg = "Empty value supplied"
                 raise sqlmapNoneDataException, errMsg
         else:
-            checkDynamicContent(firstPage, secondPage)
+            conf.seqMatcher.set_seq1(firstPage)
+            conf.seqMatcher.set_seq2(secondPage)
 
-            count = 0
-            while not Request.queryPage():
-                count += 1
+            # In case of an intolerable difference turn on dynamicity removal engine
+            if conf.seqMatcher.quick_ratio() <= UPPER_RATIO_BOUND:
+                checkDynamicContent(firstPage, secondPage)
 
-                if count > conf.retries:
-                    errMsg = "target url is too dynamic. unable to continue. "
-                    errMsg += "consider using other switches (e.g. "
-                    errMsg += "--longest-common, --string, --text-only, etc.)"
-                    raise sqlmapSiteTooDynamic, errMsg
+                count = 0
+                while not Request.queryPage():
+                    count += 1
 
-                warnMsg = "target url is heavily dynamic"
-                warnMsg += ", sqlmap is going to retry the request"
-                logger.critical(warnMsg)
+                    if count > conf.retries:
+                        errMsg = "target url is too dynamic. unable to continue. "
+                        errMsg += "consider using other switches (e.g. "
+                        errMsg += "--longest-common, --string, --text-only, etc.)"
+                        raise sqlmapSiteTooDynamic, errMsg
 
-                secondPage, _ = Request.queryPage(content=True)
-                checkDynamicContent(firstPage, secondPage)
+                    warnMsg = "target url is heavily dynamic"
+                    warnMsg += ", sqlmap is going to retry the request"
+                    logger.critical(warnMsg)
+
+                    secondPage, _ = Request.queryPage(content=True)
+                    checkDynamicContent(firstPage, secondPage)
 
     return kb.pageStable
 
