Issue2681: the literal 0o8 was wrongly accepted, and evaluated as float(0.0).

amaury.forgeotdarc · amaury.forgeotdarc · commit 18a99d2e7c4d · 2008-04-24T18:07:05.000Z
This happened only when 8 is the first digit. Credits go to Lukas Meuser. git-svn-id: http://svn.python.org/projects/python/trunk@62480 6015fed2-1504-0410-9fe1-9d1591cc4771
diff --git a/Lib/test/test_compile.py b/Lib/test/test_compile.py
@@ -183,7 +183,7 @@ def test_literals_with_leading_zeroes(self):
         for arg in ["077787", "0xj", "0x.", "0e",  "090000000000000",
                     "080000000000000", "000000000000009", "000000000000008",
                     "0b42", "0BADCAFE", "0o123456789", "0b1.1", "0o4.2",
-                    "0b101j2", "0o153j2", "0b100e1", "0o777e1"]:
+                    "0b101j2", "0o153j2", "0b100e1", "0o777e1", "0o8", "0o78"]:
             self.assertRaises(SyntaxError, eval, arg)
 
         self.assertEqual(eval("0777"), 511)
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -12,6 +12,9 @@ What's New in Python 2.6 alpha 3?
 Core and builtins
 -----------------
 
+- Issue #2681: The octal literal ``0o8`` was incorrecly acctepted. Now it
+  properly raises a SyntaxError.
+
 - Patch #2617: Reserved -J and -X arguments for Jython, IronPython and other
   implementations of Python. 
 
diff --git a/Parser/tokenizer.c b/Parser/tokenizer.c
@@ -1351,7 +1351,7 @@ tok_get(register struct tok_state *tok, char **p_start, char **p_end)
                         else if (c == 'o' || c == 'O') {
 				/* Octal */
 				c = tok_nextc(tok);
-				if (c < '0' || c > '8') {
+				if (c < '0' || c >= '8') {
 					tok->done = E_TOKEN;
 					tok_backup(tok, c);
 					return ERRORTOKEN;
