diff options
author | Antoine Pitrou <solipsis@pitrou.net> | 2012-02-21 18:03:47 (GMT) |
---|---|---|
committer | Antoine Pitrou <solipsis@pitrou.net> | 2012-02-21 18:03:47 (GMT) |
commit | 86838b02f04a01445778584afb7acd0b05081335 (patch) | |
tree | e70c00698519271b4431d7aed8a339228b372f9f /Modules/main.c | |
parent | dbd3f6173a0dd15c97b60be56cc37ff1cf5487e5 (diff) | |
download | cpython-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.c | 30 |
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 */ |