imp_load_module(): correct and comment the sense of the test for '+'
in the mode (it's forbidden).
diff --git a/Python/import.c b/Python/import.c
index 112f7f6..8c9cd2f 100644
--- a/Python/import.c
+++ b/Python/import.c
@@ -2408,11 +2408,16 @@
&name, &fob, &pathname,
&suffix, &mode, &type))
return NULL;
- if (*mode &&
- !(*mode == 'r' || *mode == 'U' || strchr(mode, '+'))) {
+ if (*mode) {
+ /* Mode must start with 'r' or 'U' and must not contain '+'.
+ Implicit in this test is the assumption that the mode
+ may contain other modifiers like 'b' or 't'. */
+
+ if (!(*mode == 'r' || *mode == 'U') || strchr(mode, '+')) {
PyErr_Format(PyExc_ValueError,
"invalid file open mode %.200s", mode);
return NULL;
+ }
}
if (fob == Py_None)
fp = NULL;