[issue36272] Recursive logging crashes Interpreter in Python 3

Change by Vinay Sajip : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: -Python 3.6 ___ Python tracker ___ __

[issue36272] Recursive logging crashes Interpreter in Python 3

Vinay Sajip added the comment: New changeset 6a7a9f1d83cef628d2bacd71ee568b93f53fd6b4 by Vinay Sajip (Miss Islington (bot)) in branch '3.7': bpo-36272: Logging now propagates RecursionError (GH-12312) (GH-12391) https://github.com/python/cpython/commit/6a7a9f1d83cef628d2bacd71ee568b93f53fd6b4

[issue36272] Recursive logging crashes Interpreter in Python 3

Change by miss-islington : -- pull_requests: +12346 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue36272] Recursive logging crashes Interpreter in Python 3

Vinay Sajip added the comment: New changeset 65f64b1903ae85b97a30f514bbc1b7ce940c3af2 by Vinay Sajip (Rémi Lapeyre) in branch 'master': bpo-36272: Logging now propagates RecursionError (GH-12312) https://github.com/python/cpython/commit/65f64b1903ae85b97a30f514bbc1b7ce940c3af2 -- __

[issue36272] Recursive logging crashes Interpreter in Python 3

Change by miss-islington : -- pull_requests: +12307 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue36272] Recursive logging crashes Interpreter in Python 3

Rémi Lapeyre added the comment: Hi Brett, I confirm the test case breaks both Python3.7 and 3.8. I opened a PR to fix the problem. -- versions: +Python 3.7, Python 3.8 ___ Python tracker

[issue36272] Recursive logging crashes Interpreter in Python 3

Change by Rémi Lapeyre : -- keywords: +patch pull_requests: +12287 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-lis

[issue36272] Recursive logging crashes Interpreter in Python 3

Brett Cannon added the comment: Limiting the version scope to 3.6 until someone reproduces on 3.7 and/or 3.8. -- nosy: +brett.cannon versions: -Python 3.7, Python 3.8 ___ Python tracker

[issue36272] Recursive logging crashes Interpreter in Python 3

Karthikeyan Singaravelan added the comment: > Stack exhaustion doesn't seem to be due to be the root cause. A simple > recursive function doesn't crash the interpreter in Python 3.6. Yes, sorry I got misleaded. I have added logging module author, @vinay.sajip to the issue. -- _

[issue36272] Recursive logging crashes Interpreter in Python 3

Rémi Lapeyre added the comment: The following patch fixes the issue and raise RecursionError as expecting without core dump: diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py index b4659af7cc..7457549cb9 100644 --- a/Lib/logging/__init__.py +++ b/Lib/logging/__init__.py @@ -1094,6

[issue36272] Recursive logging crashes Interpreter in Python 3

Rémi Lapeyre added the comment: I'm not sure issue35542. I think this happens because while logging the recursion limit is hit which calls https://github.com/python/cpython/blob/master/Python/ceval.c#L535-L539. The RecursionError is then handled by https://github.com/python/cpython/blob/mast

[issue36272] Recursive logging crashes Interpreter in Python 3

Saim Raza added the comment: Stack exhaustion doesn't seem to be due to be the root cause. A simple recursive function doesn't crash the interpreter in Python 3.6. >>> def rec(): rec() >>> rec() Traceback (most recent call last): File "", line 1, in File "", line 1, in rec File "", lin

[issue36272] Recursive logging crashes Interpreter in Python 3

Change by Karthikeyan Singaravelan : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue36272] Recursive logging crashes Interpreter in Python 3

Karthikeyan Singaravelan added the comment: I am not sure this is related to logging and looks similar to issue35542 except stack (depends on OS) is exhausted without setrecursionlimit(). What does below return? def rec(): rec() rec() -- nosy: +xtreak _

[issue36272] Recursive logging crashes Interpreter in Python 3

New submission from Saim Raza : Following code logs an error and calls itself leading to stack overflow and eventually core dump in Python 3.6. >>> import logging >>> def rec(): ... logging.error("foo") ... rec() >>> rec() [1]101641 abort (core dumped) python3 FTR, this doesn't c