minor fix (it was quite ... to have this check at the later stage)

stamparm · stamparm · commit b327bbcd9ba6 · 2011-04-06T08:39:24.000Z
diff --git a/lib/core/option.py b/lib/core/option.py
@@ -1496,6 +1496,10 @@ def __basicOptionValidation():
         errMsg = "value for --time-sec option must be an integer greater than 0"
         raise sqlmapSyntaxException, errMsg
 
+    if isinstance(conf.uCols, basestring) and ("-" not in conf.uCols or len(conf.uCols.split("-")) != 2):
+        errMsg = "--union-cols must be a range with hyphon (e.g. 1-10)"
+        raise sqlmapSyntaxException, errMsg
+
 def init(inputOptions=advancedDict(), overrideOptions=False):
     """
     Set attributes into both configuration and knowledge base singletons
diff --git a/lib/techniques/inband/union/use.py b/lib/techniques/inband/union/use.py
@@ -80,9 +80,6 @@ def __configUnionChar(char):
             conf.uChar = "'%s'" % char
 
     def __configUnionCols(columns):
-        if "-" not in columns or len(columns.split("-")) != 2:
-            raise sqlmapSyntaxException, "--union-cols must be a range with hyphon (e.g. 1-10)"
-
         columns = columns.replace(" ", "")
         colsStart, colsStop = columns.split("-")
 
