Mailing List Archive: Python: Bugs

[issue15604] PyObject_IsTrue failure checks

Index | Next | Previous | View Threaded

report at bugs

Aug 9, 2012, 8:24 AM

Post #1 of 11 (151 views)
Permalink

[issue15604] PyObject_IsTrue failure checks

New submission from Serhiy Storchaka:

PyObject_IsTrue can fail, but not everywhere in a code a returned value checked. Here is a patches which add such checks.

Note, patches for all three Python versions are rather different.

----------
components: Interpreter Core, Library (Lib)
files: istrue_check-3.3.patch
keywords: needs review, patch
messages: 167789
nosy: storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: PyObject_IsTrue failure checks
type: resource usage
versions: Python 2.7, Python 3.2, Python 3.3
Added file: http://bugs.python.org/file26742/istrue_check-3.3.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 9, 2012, 8:25 AM

Post #2 of 11 (155 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Changes by Serhiy Storchaka <storchaka [at] gmail>:

Added file: http://bugs.python.org/file26743/istrue_check-3.2.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 9, 2012, 8:25 AM

Post #3 of 11 (143 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Changes by Serhiy Storchaka <storchaka [at] gmail>:

Added file: http://bugs.python.org/file26744/istrue_check-2.7.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 9, 2012, 10:04 AM

Post #4 of 11 (145 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Antoine Pitrou added the comment:

Is it possible to add test cases for (at least some of) these issues?

----------
nosy: +pitrou

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 10, 2012, 11:37 AM

Post #5 of 11 (138 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Serhiy Storchaka added the comment:

I shall try to do this, but it will take a lot of time. Besides, now in the code there are a lot of *correct* checked usage of PyObject_IsTrue without test cases. So I'm not sure that the tests are needed here, and that they are worth the effort.

----------

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 13, 2012, 5:56 AM

Post #6 of 11 (138 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Changes by Andrew Svetlov <andrew.svetlov [at] gmail>:

----------
nosy: +asvetlov

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 14, 2012, 2:45 PM

Post #7 of 11 (131 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Serhiy Storchaka added the comment:

Patches updated to reflect Antoine's comments.

----------
Added file: http://bugs.python.org/file26813/istrue_check-3.3_2.patch
Added file: http://bugs.python.org/file26814/istrue_check-3.2_2.patch
Added file: http://bugs.python.org/file26815/istrue_check-2.7_2.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________

Attachments:

istrue_check-3.3_2.patch (11.5 KB)

istrue_check-3.2_2.patch (8.77 KB)

istrue_check-2.7_2.patch (11.1 KB)

report at bugs

Aug 15, 2012, 1:56 PM

Post #8 of 11 (130 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Serhiy Storchaka added the comment:

Patches updated again.

----------
Added file: http://bugs.python.org/file26838/istrue_check-3.3_3.patch
Added file: http://bugs.python.org/file26839/istrue_check-3.2_3.patch
Added file: http://bugs.python.org/file26840/istrue_check-2.7_3.patch

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________

Attachments:

istrue_check-3.3_3.patch (11.9 KB)

istrue_check-3.2_3.patch (9.22 KB)

istrue_check-2.7_3.patch (11.5 KB)

report at bugs

Aug 15, 2012, 2:27 PM

Post #9 of 11 (134 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Roundup Robot added the comment:

New changeset ba1c48f8b571 by Antoine Pitrou in branch '2.7':
Issue #15604: Update uses of PyObject_IsTrue() to check for and handle errors correctly.
http://hg.python.org/cpython/rev/ba1c48f8b571

New changeset 56dc7b09f390 by Antoine Pitrou in branch '3.2':
Issue #15604: Update uses of PyObject_IsTrue() to check for and handle errors correctly.
http://hg.python.org/cpython/rev/56dc7b09f390

New changeset b878df1d23b1 by Antoine Pitrou in branch 'default':
Issue #15604: Update uses of PyObject_IsTrue() to check for and handle errors correctly.
http://hg.python.org/cpython/rev/b878df1d23b1

----------
nosy: +python-dev

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 15, 2012, 2:45 PM

Post #10 of 11 (130 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Serhiy Storchaka added the comment:

Thank you Antoine.

----------
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

Aug 15, 2012, 6:07 PM

Post #11 of 11 (130 views)
Permalink

[issue15604] PyObject_IsTrue failure checks [In reply to]

Changes by Jesús Cea Avión <jcea [at] jcea>:

----------
nosy: +jcea

_______________________________________
Python tracker <report [at] bugs>
<http://bugs.python.org/issue15604>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads