FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-106030: Miscellaneous fixes in Python/suggestions.c (GH-106031) by miss-islington · Pull Request #106036 · python/cpython · GitHub

Repository navigation

[3.12] gh-106030: Miscellaneous fixes in Python/suggestions.c (GH-106031) - #106036

Merged
hauntsaninja merged 1 commit into
python:3.12from
miss-islington:backport-c8c162e-3.12
Jun 27, 2023
Merged

hauntsaninja merged 1 commit into
python:3.12from
miss-islington:backport-c8c162e-3.12

Conversation

miss-islington commented Jun 23, 2023 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor
  • PyDict_GetItem() and PyObject_HasAttr() suppress arbitrary errors and
    should not be used.
  • PyUnicode_CompareWithASCIIString() only works if the second argument
    is ASCII string.
  • Refleak in get_suggestions_for_name_error.
  • Use of borrowed pointer after possible freeing (self).
  • Add some missing error checks.
    (cherry picked from commit c8c162e)

Co-authored-by: Serhiy Storchaka storchaka@gmail.com

…H-106031)

* PyDict_GetItem() and PyObject_HasAttr() suppress arbitrary errors and
  should not be used.
* PyUnicode_CompareWithASCIIString() only works if the second argument
  is ASCII string.
* Refleak in get_suggestions_for_name_error.
* Use of borrowed pointer after possible freeing (self).
* Add some missing error checks.
(cherry picked from commit c8c162e)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
hauntsaninja merged commit 1bb2bf7 into python:3.12 Jun 27, 2023
miss-islington deleted the backport-c8c162e-3.12 branch June 27, 2023 03:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL