From: Jeremy Hylton <jeremy@alum.mit.edu>
Date: Fri, 9 Nov 2001 19:50:08 +0000 (+0000)
Subject: Fix SF buf #480096: Assign to __debug__ still allowed
X-Git-Tag: v2.2.1c1~805
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=778e26546284f956cbf8a85a8b7a0bf28c194410;p=python

Fix SF buf #480096: Assign to __debug__ still allowed

Easy enough to catch assignment in the compiler.  The perverse user
can still change the value of __debug__, but that may be the least he
can do.
---

diff --git a/Lib/test/test_compile.py b/Lib/test/test_compile.py
index 9f20ba1377..e976a30dd5 100644
--- a/Lib/test/test_compile.py
+++ b/Lib/test/test_compile.py
@@ -1,5 +1,16 @@
 from test_support import verbose, TestFailed
 
+if verbose:
+    print "Testing whether compiler catches assignment to __debug__"
+
+try:
+    compile('__debug__ = 1', '?', 'single')
+except SyntaxError:
+    pass
+
+import __builtin__
+setattr(__builtin__, '__debug__', 'sure')
+
 if verbose:
     print 'Running tests on argument handling'
 
@@ -21,7 +32,8 @@ try:
 except SyntaxError:
     pass
 
-print "testing complex args"
+if verbose:
+    print "testing complex args"
 
 def comp_args((a, b)):
     print a,b
diff --git a/Python/compile.c b/Python/compile.c
index c8a56699bc..1a46064dd4 100644
--- a/Python/compile.c
+++ b/Python/compile.c
@@ -5459,8 +5459,13 @@ symtable_assign(struct symtable *st, node *n, int def_flag)
 			n = CHILD(n, 1);
 			goto loop;
 		} else if (TYPE(tmp) == NAME) {
-			if (strcmp(STR(tmp), "__debug__") == 0)
-				symtable_warn(st, ASSIGN_DEBUG);
+			if (strcmp(STR(tmp), "__debug__") == 0) {
+				PyErr_SetString(PyExc_SyntaxError, 
+						ASSIGN_DEBUG);
+				PyErr_SyntaxLocation(st->st_filename,
+					     st->st_cur->ste_opt_lineno);
+				st->st_errors++;
+			}
 			symtable_add_def(st, STR(tmp), DEF_LOCAL | def_flag);
 		}
 		return;