Proper fix for --start and --stop consistency amongst different techniques

bdamele · bdamele · commit 938716e361bb · 2011-07-26T10:06:28.000Z
diff --git a/lib/core/option.py b/lib/core/option.py
@@ -1700,8 +1700,8 @@ def __basicOptionValidation():
         raise sqlmapSyntaxException, errMsg
 
     if conf.limitStart is not None and isinstance(conf.limitStart, int) and conf.limitStart > 0 and \
-       conf.limitStop is not None and isinstance(conf.limitStop, int) and conf.limitStop <= conf.limitStart:
-        errMsg = "value for --start (limitStart) option must be smaller than value for --stop (limitStop) option"
+       conf.limitStop is not None and isinstance(conf.limitStop, int) and conf.limitStop < conf.limitStart:
+        errMsg = "value for --start (limitStart) option must be smaller or equal than value for --stop (limitStop) option"
         raise sqlmapSyntaxException, errMsg
 
     if conf.firstChar is not None and isinstance(conf.firstChar, int) and conf.firstChar > 0 and \
diff --git a/lib/techniques/error/use.py b/lib/techniques/error/use.py
@@ -283,7 +283,7 @@ def errorUse(expression, expected=None, resumeValue=True, dump=False):
                     stopLimit += startLimit
             elif dump:
                 if conf.limitStart:
-                    startLimit = conf.limitStart
+                    startLimit = conf.limitStart - 1
                 if conf.limitStop:
                     stopLimit = conf.limitStop
 
diff --git a/lib/techniques/union/use.py b/lib/techniques/union/use.py
@@ -221,7 +221,7 @@ def unionUse(expression, unpack=True, dump=False):
                     stopLimit += startLimit
             elif dump:
                 if conf.limitStart:
-                    startLimit = conf.limitStart
+                    startLimit = conf.limitStart - 1
                 if conf.limitStop:
                     stopLimit = conf.limitStop
 
