Skip to content

bpo-1635741: Enable unicode_release_interned() without insure or valgrind. #21087

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
shihai1991 wants to merge 9 commits into from
Closed

bpo-1635741: Enable unicode_release_interned() without insure or valgrind. #21087

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ed3cd98
Enable unicode_release_interned() without insure or valgrind.
shihai1991 Jun 23, 2020
3c8db50
resolve use after free
shihai1991 Jun 23, 2020
03f650d
Merge branch 'master' into bpo_1635741_interned_unicode
shihai1991 Jun 23, 2020
b2e578d
resolve confilicts
shihai1991 Jun 24, 2020
4618783
Use PyDict_Next() in unicode_release_interned()
shihai1991 Jun 25, 2020
8b47de1
resolve conflicts
shihai1991 Jun 25, 2020
33272ac
Merge remote-tracking branch 'origin/master' into bpo_1635741_interne…
shihai1991 Jun 25, 2020
ec1671b
update vars' name
shihai1991 Jun 25, 2020
b0da3bd
_PyUnicode_Fini() before _PyDict_Fini()
shihai1991 Jul 1, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Enable unicode_release_interned() without insure or valgrind.
  • Loading branch information
@shihai1991
shihai1991 committed Jun 23, 2020
commit ed3cd983f77707ddbb3cc5df8041e4bb1068c9cb
13 changes: 0 additions & 13 deletions Objects/unicodeobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15657,7 +15657,6 @@ PyUnicode_InternFromString(const char *cp)
}


#if defined(WITH_VALGRIND) || defined(__INSURE__)
static void
unicode_release_interned(void)
{
Expand Down Expand Up @@ -15715,7 +15714,6 @@ unicode_release_interned(void)
PyDict_Clear(interned);
Py_CLEAR(interned);
}
#endif


/********************* Unicode Iterator **************************/
Expand Down Expand Up @@ -16206,18 +16204,7 @@ void
_PyUnicode_Fini(PyThreadState *tstate)
{
if (_Py_IsMainInterpreter(tstate)) {
#if defined(WITH_VALGRIND) || defined(__INSURE__)
/* Insure++ is a memory analysis tool that aids in discovering
* memory leaks and other memory problems. On Python exit, the
* interned string dictionaries are flagged as being in use at exit
* (which it is). Under normal circumstances, this is fine because
* the memory will be automatically reclaimed by the system. Under
* memory debugging, it's a huge source of useless noise, so we
* trade off slower shutdown for less distraction in the memory
* reports. -baw
*/
unicode_release_interned();
#endif /* __INSURE__ */

Py_CLEAR(unicode_empty);

Expand Down