summaryrefslogtreecommitdiffstats
path: root/Modules/main.c
diff options
context:
space:
mode:
authorAntoine Pitrou <solipsis@pitrou.net>2012-02-21 18:03:47 (GMT)
committerAntoine Pitrou <solipsis@pitrou.net>2012-02-21 18:03:47 (GMT)
commit86838b02f04a01445778584afb7acd0b05081335 (patch)
treee70c00698519271b4431d7aed8a339228b372f9f /Modules/main.c
parentdbd3f6173a0dd15c97b60be56cc37ff1cf5487e5 (diff)
downloadcpython-86838b02f04a01445778584afb7acd0b05081335.zip
cpython-86838b02f04a01445778584afb7acd0b05081335.tar.gz
cpython-86838b02f04a01445778584afb7acd0b05081335.tar.bz2
Fix test failure in test_cmd_line by initializing the hash secret at the earliest point.
Diffstat (limited to 'Modules/main.c')
-rw-r--r--Modules/main.c30
1 files changed, 28 insertions, 2 deletions
diff --git a/Modules/main.c b/Modules/main.c
index ed84aa0..6b2d0f5 100644
--- a/Modules/main.c
+++ b/Modules/main.c
@@ -337,7 +337,33 @@ Py_Main(int argc, wchar_t **argv)
orig_argc = argc; /* For Py_GetArgcArgv() */
orig_argv = argv;
+ /* Hash randomization needed early for all string operations
+ (including -W and -X options). */
+ while ((c = _PyOS_GetOpt(argc, argv, PROGRAM_OPTS)) != EOF) {
+ if (c == 'm' || c == 'c') {
+ /* -c / -m is the last option: following arguments are
+ not interpreter options. */
+ break;
+ }
+ switch (c) {
+ case 'E':
+ Py_IgnoreEnvironmentFlag++;
+ break;
+ case 'R':
+ Py_HashRandomizationFlag++;
+ break;
+ }
+ }
+ /* The variable is only tested for existence here; _PyRandom_Init will
+ check its value further. */
+ if (!Py_HashRandomizationFlag &&
+ (p = Py_GETENV("PYTHONHASHSEED")) && *p != '\0')
+ Py_HashRandomizationFlag = 1;
+
+ _PyRandom_Init();
+
PySys_ResetWarnOptions();
+ _PyOS_ResetGetOpt();
while ((c = _PyOS_GetOpt(argc, argv, PROGRAM_OPTS)) != EOF) {
if (c == 'c') {
@@ -398,7 +424,7 @@ Py_Main(int argc, wchar_t **argv)
break;
case 'E':
- Py_IgnoreEnvironmentFlag++;
+ /* Already handled above */
break;
case 't':
@@ -440,7 +466,7 @@ Py_Main(int argc, wchar_t **argv)
break;
case 'R':
- Py_HashRandomizationFlag++;
+ /* Already handled above */
break;
/* This space reserved for other options */