From 4114a4afecdf1753b1140059d71bb03b4cec3ef1 Mon Sep 17 00:00:00 2001 From: Guido van Rossum Date: Thu, 18 Oct 2001 18:49:37 +0000 Subject: Fix the frozen bytecode for __hello__ (betcha didn't know that existed :-). Add a test that prevents the __hello__ bytecode from going stale unnoticed again. The test also tests the loophole noted in SF bug #404545. This test will fail right now; I'll check in the fix in a minute. --- Lib/__phello__.foo.py | 1 + Lib/test/output/test_frozen | 4 ++++ Lib/test/test_frozen.py | 26 ++++++++++++++++++++++++++ Python/frozen.c | 17 ++++++++++++----- 4 files changed, 43 insertions(+), 5 deletions(-) create mode 100644 Lib/__phello__.foo.py create mode 100644 Lib/test/output/test_frozen create mode 100644 Lib/test/test_frozen.py diff --git a/Lib/__phello__.foo.py b/Lib/__phello__.foo.py new file mode 100644 index 0000000..8e8623e --- /dev/null +++ b/Lib/__phello__.foo.py @@ -0,0 +1 @@ +# This file exists as a helper for the test.test_frozen module. diff --git a/Lib/test/output/test_frozen b/Lib/test/output/test_frozen new file mode 100644 index 0000000..76f17db --- /dev/null +++ b/Lib/test/output/test_frozen @@ -0,0 +1,4 @@ +test_frozen +Hello world... +Hello world... +Hello world... diff --git a/Lib/test/test_frozen.py b/Lib/test/test_frozen.py new file mode 100644 index 0000000..3aa91ab --- /dev/null +++ b/Lib/test/test_frozen.py @@ -0,0 +1,26 @@ +# Test the frozen module defined in frozen.c. + +from test_support import TestFailed +import sys, os + +try: + import __hello__ +except ImportError, x: + raise TestFailed, "import __hello__ failed:", x + +try: + import __phello__ +except ImportError, x: + raise TestFailed, "import __phello__ failed:", x + +try: + import __phello__.spam +except ImportError, x: + raise TestFailed, "import __phello__.spam failed:", x + +try: + import __phello__.foo +except ImportError: + pass +else: + raise TestFailed, "import __phello__.foo should have failed" diff --git a/Python/frozen.c b/Python/frozen.c index 21a0b52..e40d5d9 100644 --- a/Python/frozen.c +++ b/Python/frozen.c @@ -7,21 +7,28 @@ define a single frozen module, __hello__. Loading it will print some famous words... */ +/* To regenerate this data after the bytecode or marshal format has changed, + go to ../Tools/freeze/ and freeze the hello.py file; then copy and paste + the appropriate bytes from M___main__.c. */ + static unsigned char M___hello__[] = { 99,0,0,0,0,1,0,0,0,115,15,0,0,0,127,0, 0,127,1,0,100,0,0,71,72,100,1,0,83,40,2,0, 0,0,115,14,0,0,0,72,101,108,108,111,32,119,111,114, 108,100,46,46,46,78,40,0,0,0,0,40,0,0,0,0, - 115,8,0,0,0,104,101,108,108,111,46,112,121,115,1,0, - 0,0,63,1,0,115,0,0,0,0, + 40,0,0,0,0,40,0,0,0,0,115,8,0,0,0,104, + 101,108,108,111,46,112,121,115,1,0,0,0,63,1,0,115, + 0,0,0,0, }; +#define SIZE sizeof(M___hello__) + static struct _frozen _PyImport_FrozenModules[] = { /* Test module */ - {"__hello__", M___hello__, 90}, + {"__hello__", M___hello__, SIZE}, /* Test package (negative size indicates package-ness) */ - {"__phello__", M___hello__, -90}, - {"__phello__.spam", M___hello__, 90}, + {"__phello__", M___hello__, -SIZE}, + {"__phello__.spam", M___hello__, SIZE}, {0, 0, 0} /* sentinel */ }; -- cgit v0.12