[issue24066] send_message should take all the addresses in the To: header into account
Kirill Elagin added the comment: Ah, I’m so dumb. Of course the tests work as there are multiple addresses but still just one field. Here is the test for multiple fields. -- Added file: http://bugs.python.org/file39263/multiple_fields_test.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24066 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24105] Use after free during json encoding a dict (3)
Serhiy Storchaka added the comment: Proposed patch fixes three related issues: issue24094, issue24095 and issue24105. -- assignee: - serhiy.storchaka keywords: +patch stage: - patch review versions: +Python 3.5 Added file: http://bugs.python.org/file39264/json_encode_mutated_dict.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24105 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24112] %b does not work, as a binary output formatter
New submission from Bob Stein: `%b%42` produces a ValueError exception instead of outputting '101010' Details here: http://stackoverflow.com/a/29997703/673991 -- messages: 242388 nosy: BobStein priority: normal severity: normal status: open title: %b does not work, as a binary output formatter type: behavior versions: Python 2.7 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24112 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24112] %b does not work, as a binary output formatter
Bob Stein added the comment: Ah, you're right, my mistake. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24112 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24111] Valgrind suppression file should be updated
New submission from Antony Lee: Since PEP445, the suppressions should target _PyObject_{Free,Realloc} instead of PyObject_{Free,Realloc}. -- messages: 242382 nosy: Antony.Lee priority: normal severity: normal status: open title: Valgrind suppression file should be updated versions: Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24111 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24105] Use after free during json encoding a dict (3)
paul added the comment: @Serhiy: Not all of my bugs are in the same module. Sure, I will group them by module in the future. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24105 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24112] %b does not work, as a binary output formatter
Steven D'Aprano added the comment: This is not a bug, since %b is not supported in Python 2, only in Python 3: https://docs.python.org/2/library/stdtypes.html#string-formatting-operations No new features will be added to 2.7, so if you need %b you can use Python 3, or in Python 2.7 you can use the newer string format method instead: py {:b}.format(42) '101010' -- nosy: +steven.daprano resolution: - not a bug stage: - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24112 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Petr Viktorin added the comment: Not true (on 3.3 2.7). import sys import x Traceback (most recent call last): File stdin, line 1, in module File /tmp/x.py, line 1, in module import y File /tmp/y.py, line 1, in module 1/0 ZeroDivisionError: division by zero sys.modules['x'] Traceback (most recent call last): File stdin, line 1, in module KeyError: 'x' sys.modules['y'] Traceback (most recent call last): File stdin, line 1, in module KeyError: 'y' For 3.4, PEP 451 explicitly specifies ``del sys.modules[spec.name]`` if ``loader.exec_module`` fails [0]. [0] https://www.python.org/dev/peps/pep-0451/#how-loading-will-work -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Walter Dörwald added the comment: The patch does indeed fix the segmentation fault. However the exception message looks confusing: TypeError: don't know how to handle UnicodeEncodeError in error callback -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24066] send_message should take all the addresses in the To: header into account
R. David Murray added the comment: Ah, but that is intentional. Those fields can only appear once per message, per the RFC. The new email API will raise an error if you attempt to add them more than once. Perhaps we should raise an error instead of ignoring the duplicates, given that we are actively inspecting those headers. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24066 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Serhiy Storchaka added the comment: Py_TYPE() is necessary when the argument is not of type PyObject* (e.g. PyUnicodeObject*). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24109] Documentation for difflib uses optparse
Keith Gray added the comment: I took a look at Tools/scripts/diff.py and it looks like it got converted to use argparse 9 months ago. I think I should be able to just include that in the difflib documentation directly. My next question is which branches need to have this changed? Do I just make the documentation change to the default branch or do I need to change it in 2.7, 3.2, 3.3, and 3.4 as well? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24109 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24109] Documentation for difflib uses optparse
Keith Gray added the comment: Here is the patch. I have tested it against tip. Let me know if you need anything else. -- keywords: +patch Added file: http://bugs.python.org/file39268/issue-24109.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24109 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24110] zipfile.ZipFile.write() does not accept bytes arcname
R. David Murray added the comment: Ah, I *thought* there was an issue for that, but I didn't find it when I searched. So this is just a doc issue to fix the docs to reflect current reality. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24110 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Serhiy Storchaka added the comment: Here is a patch that makes error message consistent with type checking. -- Added file: http://bugs.python.org/file39266/codecs_error_handlers_issubclass_2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Changes by Serhiy Storchaka storch...@gmail.com: -- keywords: +patch Added file: http://bugs.python.org/file39265/codecs_error_handlers_issubclass.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Serhiy Storchaka added the comment: Here is simpler reproducer: import codecs class X(str): __class__ = UnicodeEncodeError codecs.ignore_errors(X()) The problem is that PyObject_IsInstance() is fooled by custom __class__, but then builtin error handlers handle error object as having UnicodeEncodeError layout, while it doesn't. Here is a patch that fixes the issue by using PyObject_IsSubclass() of exc-ob_type instead of PyObject_IsInstance(). -- assignee: - serhiy.storchaka stage: needs patch - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24102] Multiple type confusions in unicode error handlers
Walter Dörwald added the comment: Looks much better. However shouldn't: exc-ob_type-tp_name be: Py_TYPE(exc)-tp_name (although there are still many spots in the source that still use ob_type-tp_name) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24102 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24113] shlex constructor unreachable code
New submission from Michael Smith: In its __init__ method, shlex.shlex sets self.debug = 0. An `if self.debug:` statement follows shortly thereafter and without allowing the user to change self.debug. The code inside the if statement is unreachable. Users should either be permitted to set debug on via an optional __init__ parameter, or debug should be taken out of the initializer altogether. -- components: Library (Lib) messages: 242399 nosy: Michael.Smith priority: normal severity: normal status: open title: shlex constructor unreachable code type: behavior versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python 3.5, Python 3.6 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24113 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24113] shlex constructor unreachable code
Michael Smith added the comment: Hat tip abarnert on StackOverflow for digging in. http://stackoverflow.com/questions/29996208/putting-shlex-in-debug-mode This code was introduced in https://hg.python.org/cpython/rev/81a121d21340 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24113 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24066] send_message should take all the addresses in the To: header into account
Kirill Elagin added the comment: Oh, I see now. It is a good idea to raise an error either in `send_message` or at the moment when a second `To`/`Cc`/`Bcc` header is added to the message. -- resolution: - not a bug status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24066 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24114] ctypes.utils uninitialized variable 'path'
New submission from Kees Bos: In certain corner cases, the ctypes.util can raise an error for a uninitialized variable 'path' when with sunos5 the clre program exists, but fails to return valid output lines. (Also in 2.7.10rc0) -- components: ctypes files: ctypes.util-path.patch keywords: patch messages: 242401 nosy: kees priority: normal severity: normal status: open title: ctypes.utils uninitialized variable 'path' type: crash versions: Python 2.7 Added file: http://bugs.python.org/file39267/ctypes.util-path.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24114 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24056] Expose closure generator status in function repr()
Nick Coghlan added the comment: The main reason I suggest using the postfix parenthetical syntax is to make it clear that we're exposing behavioural feature flags for a single underlying type. A prefix syntax would make them look like distinct types, which would be misleading in a different way. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24056 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24109] Documentation for difflib uses optparse
R. David Murray added the comment: Since the Tools script was only changed in 3.5 (I was looking at the wrong branch), I think the docs should only be changed for 3.5. -- stage: - commit review versions: -Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24109 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Eric Snow added the comment: Either the docs are out-of-date or they are really poorly worded. Most likely it's the former, but I'm taking a look. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Changes by R. David Murray rdmur...@bitdance.com: -- stage: - commit review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24109] Documentation for difflib uses optparse
Changes by R. David Murray rdmur...@bitdance.com: -- versions: +Python 3.5 -Python 3.6 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24109 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24115] PyObject_IsInstance() and PyObject_IsSubclass() can fail
New submission from Serhiy Storchaka: PyObject_IsInstance() and PyObject_IsSubclass() cat return 0, 1, or -1. But some code use if (PyObject_IsInstance(...)) or if (!PyObject_IsInstance(...)). This should be fixed. -- assignee: serhiy.storchaka components: Extension Modules, Interpreter Core messages: 242407 nosy: serhiy.storchaka priority: normal severity: normal stage: needs patch status: open title: PyObject_IsInstance() and PyObject_IsSubclass() can fail type: behavior versions: Python 2.7, Python 3.4, Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24115 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
R. David Murray added the comment: OK, I can't reproduce it either, neither in python3 nor python2. Brett, is this left over from a long time ago? Heh. I just tried another experiment and got an interesting result: rdmurray@pydev:~/python/p34cat temp1.py import temp2 foo = 1 rdmurray@pydev:~/python/p34cat temp2.py import temp1 badval rdmurray@pydev:~/python/p34./python Python 3.4.3+ (3.4:b53cfcfdfe47, May 2 2015, 12:51:46) [GCC 4.8.2] on linux Type help, copyright, credits or license for more information. import temp2 Traceback (most recent call last): File stdin, line 1, in module File /home/rdmurray/python/p34/temp2.py, line 3, in module badval NameError: name 'badval' is not defined import sys sorted(sys.modules.keys()) ['__main__', '_codecs', '_collections_abc', '_frozen_importlib', '_imp', '_io', '_sitebuiltins', '_stat', '_sysconfigdata', '_thread', '_warnings', '_weakref', '_weakrefset', 'abc', 'atexit', 'builtins', 'codecs', 'encodings', 'encodings.aliases', 'encodings.latin_1', 'encodings.utf_8', 'errno', 'genericpath', 'io', 'marshal', 'os', 'os.path', 'posix', 'posixpath', 'readline', 'rlcompleter', 'signal', 'site', 'stat', 'sys', 'sysconfig', 'temp1', 'zipimport'] import temp1 temp1.temp2 module 'temp2' from '/home/rdmurray/python/p34/temp2.py' sorted(sys.modules.keys()) ['__main__', '_codecs', '_collections_abc', '_frozen_importlib', '_imp', '_io', '_sitebuiltins', '_stat', '_sysconfigdata', '_thread', '_warnings', '_weakref', '_weakrefset', 'abc', 'atexit', 'builtins', 'codecs', 'encodings', 'encodings.aliases', 'encodings.latin_1', 'encodings.utf_8', 'errno', 'genericpath', 'io', 'marshal', 'os', 'os.path', 'posix', 'posixpath', 'readline', 'rlcompleter', 'signal', 'site', 'stat', 'sys', 'sysconfig', 'temp1', 'zipimport'] More or less the reverse of the passage in question. -- versions: +Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
R. David Murray added the comment: Note that calling reload on temp1.temp2 will fail with an error that temp2 is not in sys.modules. So maybe the caveat needs rewording rather than deletion. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23088] Document that PyUnicode_AsUTF8() returns a null-terminated string
R. David Murray added the comment: Added some review comments. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23088 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Eric Snow added the comment: I'm pretty sure this is the culprit: changeset: 32882:331e60d8ce6da19b168849418776fea0940787ec branch: legacy-trunk user:Tim Peters tim.pet...@gmail.com date:Mon Aug 02 03:52:12 2004 + summary: PyImport_ExecCodeModuleEx(): remove module from sys.modules in error cases. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
R. David Murray added the comment: Thanks, Petr. (And Eric.) -- resolution: - fixed stage: commit review - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24108] fnmatch.translate('*.txt') fails
R. David Murray added the comment: Thanks Christophe and Merlijn. -- nosy: +r.david.murray resolution: - fixed stage: - resolved status: open - closed versions: +Python 2.7, Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24108 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24101] Use after free in siftup
Roundup Robot added the comment: New changeset d356e68de236 by Raymond Hettinger in branch '2.7': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/d356e68de236 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24101 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24100] Use after free in siftdown (2)
Roundup Robot added the comment: New changeset d356e68de236 by Raymond Hettinger in branch '2.7': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/d356e68de236 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24100 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24099] Use after free in siftdown (1)
Roundup Robot added the comment: New changeset d356e68de236 by Raymond Hettinger in branch '2.7': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/d356e68de236 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24099 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Roundup Robot added the comment: New changeset 3cdeafd18e61 by R David Murray in branch '3.4': #24081: Remove obsolete caveat from import docs. https://hg.python.org/cpython/rev/3cdeafd18e61 New changeset d57e0c6d292d by R David Murray in branch 'default': Merge: #24081: Remove obsolete caveat from import docs. https://hg.python.org/cpython/rev/d57e0c6d292d New changeset fd1549dd8065 by R David Murray in branch '2.7': #24081: Remove obsolete caveat from import docs. https://hg.python.org/cpython/rev/fd1549dd8065 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24108] fnmatch.translate('*.txt') fails
Roundup Robot added the comment: New changeset c5c65ef84a77 by R David Murray in branch '3.4': #24108: Update fnmatch.translate example to show correct output. https://hg.python.org/cpython/rev/c5c65ef84a77 New changeset cc6aed8ecb0d by R David Murray in branch 'default': Merge: #24108: Update fnmatch.translate example to show correct output. https://hg.python.org/cpython/rev/cc6aed8ecb0d New changeset 5d356223f075 by R David Murray in branch '2.7': #24108: Update fnmatch.translate example to show correct output. https://hg.python.org/cpython/rev/5d356223f075 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24108 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24056] Expose closure generator status in function repr()
R. David Murray added the comment: Although I like the look of the repr Terry proposes better, I agree with Nick: it would imply that the types were distinct, which they are not. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24056 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24099] Use after free in siftdown (1)
Roundup Robot added the comment: New changeset 813854f49f9d by Raymond Hettinger in branch '3.4': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/813854f49f9d -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24099 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24100] Use after free in siftdown (2)
Roundup Robot added the comment: New changeset 813854f49f9d by Raymond Hettinger in branch '3.4': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/813854f49f9d -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24100 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24101] Use after free in siftup
Roundup Robot added the comment: New changeset 813854f49f9d by Raymond Hettinger in branch '3.4': Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq. https://hg.python.org/cpython/rev/813854f49f9d -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24101 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24100] Use after free in siftdown (2)
Serhiy Storchaka added the comment: It would be nice to add tests based on provided demo scripts. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24100 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Eric Snow added the comment: I've verified that the documentation is correct under Python 2.3. The behavior changed under Python 2.4 (and the docs were not updated). I expect that the change in behavior is an unintended consequence of a change in the import system for 2.4. There were 7 in 2.4, so I'm checking to see which did it. FYI, our backward-compatibility policy would preclude a change like this, which is why I think it was unintentional. That no one noticed implies that we did not have a unit test for the case and that folks really weren't relying on the behavior anyway (at least not enough to open a bug). :) I suppose there could be an old bug on this, but if that the case then someone decided not to change it back. Regardless, the docs should be fixed by removing the referenced paragraph. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Eric Snow added the comment: patch LGTM for the 3 branches. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23572] functools.singledispatch fails when not BaseClass is True
Serhiy Storchaka added the comment: An example is Enum. -- nosy: +serhiy.storchaka ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23572 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24060] Clearify necessities for logging with timestamps
Roundup Robot added the comment: New changeset 5ce760e2cc59 by Vinay Sajip in branch '2.7': Issue #24060: Made logging.Formatter documentation a little clearer. https://hg.python.org/cpython/rev/5ce760e2cc59 New changeset 88c141233d1e by Vinay Sajip in branch '3.4': Issue #24060: Made logging.Formatter documentation a little clearer. https://hg.python.org/cpython/rev/88c141233d1e New changeset 1c72e9c1b3b4 by Vinay Sajip in branch 'default': Closes #24060: Merged documentation update from 3.4. https://hg.python.org/cpython/rev/1c72e9c1b3b4 -- nosy: +python-dev resolution: - fixed stage: - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24060 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Eric Snow added the comment: As I mentioned, I'm pretty sure that the failing venv test is due to the bundled pip. Here's the test output: test test_venv failed -- Traceback (most recent call last): File /home/esnow/projects/cpython/Lib/test/test_venv.py, line 356, in test_with_pip with_pip=True) subprocess.CalledProcessError: Command '['/tmp/tmphxh1zztf/bin/python', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1 During handling of the above exception, another exception occurred: Traceback (most recent call last): File /home/esnow/projects/cpython/Lib/test/test_venv.py, line 362, in test_with_pip self.fail(msg.format(exc, details)) AssertionError: Command '['/tmp/tmphxh1zztf/bin/python', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1 **Subprocess Output** Traceback (most recent call last): File /home/esnow/projects/cpython/Lib/runpy.py, line 170, in _run_module_as_main __main__, mod_spec) File /home/esnow/projects/cpython/Lib/runpy.py, line 85, in _run_code exec(code, run_globals) File /home/esnow/projects/cpython/Lib/ensurepip/__main__.py, line 4, in module ensurepip._main() File /home/esnow/projects/cpython/Lib/ensurepip/__init__.py, line 209, in _main default_pip=args.default_pip, File /home/esnow/projects/cpython/Lib/ensurepip/__init__.py, line 116, in bootstrap _run_pip(args + [p[0] for p in _PROJECTS], additional_paths) File /home/esnow/projects/cpython/Lib/ensurepip/__init__.py, line 40, in _run_pip import pip File frozen importlib._bootstrap, line 958, in _find_and_load File frozen importlib._bootstrap, line 947, in _find_and_load_unlocked File frozen importlib._bootstrap, line 655, in _load_unlocked File frozen importlib._bootstrap, line 625, in _load_backward_compatible File /tmp/tmp57j9vkrt/pip-6.1.1-py2.py3-none-any.whl/pip/__init__.py, line 13, in module File frozen importlib._bootstrap, line 958, in _find_and_load File frozen importlib._bootstrap, line 947, in _find_and_load_unlocked File frozen importlib._bootstrap, line 655, in _load_unlocked File frozen importlib._bootstrap, line 625, in _load_backward_compatible File /tmp/tmp57j9vkrt/pip-6.1.1-py2.py3-none-any.whl/pip/utils/__init__.py, line 22, in module File frozen importlib._bootstrap, line 958, in _find_and_load File frozen importlib._bootstrap, line 947, in _find_and_load_unlocked File frozen importlib._bootstrap, line 655, in _load_unlocked File frozen importlib._bootstrap, line 625, in _load_backward_compatible File /tmp/tmp57j9vkrt/pip-6.1.1-py2.py3-none-any.whl/pip/_vendor/pkg_resources/__init__.py, line 1712, in module AttributeError: module 'importlib._bootstrap' has no attribute 'SourceFileLoader' -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24068] statistics module - incorrect results with boolean input
Wolfgang Maier added the comment: uploading an alternate, possibly slightly clearer version of the patch -- Added file: http://bugs.python.org/file39269/statistics._sum.v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24068 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24091] Use after free in Element.extend (1)
Serhiy Storchaka added the comment: Here is a patch that fixes issue24091, issue24092, issue24093, and several other similar bugs. -- assignee: - serhiy.storchaka keywords: +patch stage: - patch review versions: +Python 3.5 Added file: http://bugs.python.org/file39270/etree_crashes.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24091 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue21327] socket.type value changes after using settimeout()
R. David Murray added the comment: I added some review comments. Something I don't understand: if settimeout sets NONBLOCK, and CLOEXEC is always set now, why does s=socket.socket() yield something whose type is SOCK_STREAM, while settimeout causes type to change? I don't understand the relationship between SOCK_NONBLOCK and O_NONBLOCK, either. Whoever came up with this API in linux was crazy, if you ask me. On the other hand, having type return what it does is clearly correct, given that we are mirroring the C API. Having a new sock_type attribute that provides the value without the extra flags makes sense in that context, IMO. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21327 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Eric Snow added the comment: Looks like setuptool's pkg_resources is directly importing importlib._bootstrap. I've filed a bug: https://bitbucket.org/pypa/setuptools/issue/378. In the meantime, what are our options for getting that test passing? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: Removed file: http://bugs.python.org/file39271/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Eric Snow added the comment: Here's the correct patch. -- Added file: http://bugs.python.org/file39272/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2292] Missing *-unpacking generalizations
Thomas Wouters added the comment: The latest patch looks good to me. No need to do the additional AST refactoring if it's going to make PEP 492's implementor's life harder (but I do read Guido's comment as a reason to check this in sooner rather than later :) So, unless anyone objects I'll check in the latest patch on Monday. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2292 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Nick Coghlan added the comment: As a compatibility hack for setuptools versions with the issue, I'd suggest making importlib._bootstrap.setup alias SourceFileLoader back into importlib._bootstrap, with an explanation and link to back to this issue in a comment. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24081] Obsolete caveat in reload() docs
Nick Coghlan added the comment: I vaguely recall discussing that change, and my recollection is that it was deliberate in order to evict improperly configured modules - we didn't count it as a significant compatibility break because it solely affected an error handling path. I guess we never thought to check if the old misbehaviour was formally documented. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24081 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Eric Snow added the comment: Gah. I had tried exactly that but did it in the wrong spot. Here's an updated patch which fixes the test. -- Added file: http://bugs.python.org/file39273/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24100] Use after free in siftdown (2)
Raymond Hettinger added the comment: Go ahead and add tests if you like. I don't see the point. The fix was to simply backport code that was already in Py3.5. and the code exercised by the exploit scripts is no longer there. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24100 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24045] Behavior of large returncodes (sys.exit(nn))
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- nosy: +Arfrever ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24045 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14376] sys.exit documents argument as integer but actually requires subtype of int
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- nosy: +Arfrever ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue14376 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24052] sys.exit(code) returns success to the OS for some nonzero values of code
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com: -- nosy: +Arfrever ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24052 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23955] Add python.ini file for embedded/applocal installs
Steve Dower added the comment: Added a patch that: * adds the applocal option to pyvenv.cfg * updates the docs to describe how pyvenv.cfg (home and applocal) affect sys.path * puts a default pyvenv.cfg in the ZIP distribution * adds significantly more buffer overflow protection to getpathp.c * prevents a case where sys.path may include multiple blank entries Currently none of this applies to Linux or Mac builds, though I'm led to believe there would be some value if it did. I'm not the one who can figure all the intricacies out for those platforms though. -- keywords: +patch Added file: http://bugs.python.org/file39276/23955_1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23955 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue21327] socket.type value changes after using settimeout()
R. David Murray added the comment: The behavior of s.type has changed in 3.5: it now behaves like the proposed new method behaves. Looking at the source I can't figure out why this is so. It doesn't seem as though it should be, but this needs to be sorted out before we can proceed. -- nosy: +ethan.furman ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21327 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue21327] socket.type value changes after using settimeout()
R. David Murray added the comment: Wait, I take it back. I must have run the test incorrectly. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21327 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24093] Use after free in Element.remove
Serhiy Storchaka added the comment: The patch for issue24091 fixes this issue. -- resolution: - duplicate stage: - resolved status: open - closed superseder: - Use after free in Element.extend (1) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24093 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24092] Use after free in Element.extend (2)
Serhiy Storchaka added the comment: The patch for issue24091 fixes this issue. -- resolution: - duplicate stage: needs patch - resolved status: open - closed superseder: - Use after free in Element.extend (1) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24092 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Eric Snow added the comment: Here's an updated patch with _pathy.py changed to _bootstrap_external.py (and similar changes with freezing). The patch does not include fixing the venv test (i.e. the bundled pip). Also, I'll be adding a note to NEWS. -- Added file: http://bugs.python.org/file39271/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: Removed file: http://bugs.python.org/file39273/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: Removed file: http://bugs.python.org/file39274/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24100] Use after free in siftdown (2)
Raymond Hettinger added the comment: I should add that the crasher scripts share common features and could perhaps be built into a unified test tool. Also, some care should be taken to make the tool mostly independent of non-guaranteed del behavior or CPython specific ref-counting logic to trigger a particular sequence of events. For example, pypy gives a different result than CPython 3.5 for for the various poc_sift* scripts. The reason is that the __del__ method isn't reliably triggered in a gc-only environment. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24100 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Roundup Robot added the comment: New changeset 02e3bf65b2f8 by Eric Snow in branch 'default': Issue #23911: Move path-based bootstrap code to a separate frozen module. https://hg.python.org/cpython/rev/02e3bf65b2f8 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: -- resolution: - fixed stage: patch review - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1182143] making builtin exceptions more informative
George Jenkins added the comment: Heh, just noticed this issue passed its 10 year anniversary! If someone has time to review my patch, that would be much appreciated. Thanks! -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1182143 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24094] Use after free during json encoding (PyType_IsSubtype)
Roundup Robot added the comment: New changeset 0e8c9cf1fede by Benjamin Peterson in branch '3.3': just sort the items tuple directly (closes #24094) https://hg.python.org/cpython/rev/0e8c9cf1fede New changeset f9027b10b3c4 by Benjamin Peterson in branch '3.4': merge 3.3 (#24094) https://hg.python.org/cpython/rev/f9027b10b3c4 New changeset 3bdf1816f561 by Benjamin Peterson in branch 'default': merge 3.4 (#24094) https://hg.python.org/cpython/rev/3bdf1816f561 -- nosy: +python-dev resolution: - fixed stage: needs patch - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24094 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24089] argparse crashes with AssertionError
paul j3 added the comment: It's the spaces and/or brackets in the metavar, along with a usage line that is long enough to wrap, that is raising this error. It's a known problem. http://bugs.python.org/issue11874 -- nosy: +paul.j3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24089 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: Added file: http://bugs.python.org/file39274/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23911] Move path-based bootstrap code to a separate frozen file.
Changes by Eric Snow ericsnowcurren...@gmail.com: Added file: http://bugs.python.org/file39275/path-based-importlib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23911 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2292] Missing *-unpacking generalizations
Guido van Rossum added the comment: Thanks for the review Thomas! And yes, that's what I meant. :-) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2292 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24018] add a Generator ABC
Guido van Rossum added the comment: Yeah, looks good -- Łuke, can you commit this? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24018 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue2292] Missing *-unpacking generalizations
Benjamin Peterson added the comment: It certainly would be nice to have documentation. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2292 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com