[issue39198] Lock may not be released in Logger.isEnabledFor

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

[issue39198] Lock may not be released in Logger.isEnabledFor

Vinay Sajip added the comment: New changeset d46dec981abdefba56336521c7587c8554bb1b9d by Vinay Sajip (Miss Islington (bot)) in branch '3.7': bpo-39198: Ensure logging global lock is released on exception in isEnabledFor (GH-17689) (GH-17898)

[issue39198] Lock may not be released in Logger.isEnabledFor

Vinay Sajip added the comment: New changeset bff48c6734f936257b0cfae58dbea67d43e3b245 by Vinay Sajip (Miss Islington (bot)) in branch '3.8': bpo-39198: Ensure logging global lock is released on exception in isEnabledFor (GH-17689) (GH-17897)

[issue39198] Lock may not be released in Logger.isEnabledFor

Change by miss-islington : -- pull_requests: +17308 pull_request: https://github.com/python/cpython/pull/17898 ___ Python tracker ___

[issue39198] Lock may not be released in Logger.isEnabledFor

Change by miss-islington : -- pull_requests: +17307 pull_request: https://github.com/python/cpython/pull/17897 ___ Python tracker ___

[issue39198] Lock may not be released in Logger.isEnabledFor

Vinay Sajip added the comment: New changeset 950c6795aa0ffa85e103a13e7a04e08cb34c66ad by Vinay Sajip (Derek Brown) in branch 'master': bpo-39198: Ensure logging global lock is released on exception in isEnabledFor (GH-17689)

[issue39198] Lock may not be released in Logger.isEnabledFor

Change by Ned Deily : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39198] Lock may not be released in Logger.isEnabledFor

Derek Brown added the comment: PR is here: https://github.com/python/cpython/pull/17689 -- ___ Python tracker ___ ___

[issue39198] Lock may not be released in Logger.isEnabledFor

New submission from Derek Brown : If an exception were to be thrown in a particular block of code (say, by asyncio timeouts or stopit) within the `isEnabledFor` function of `logging`, the `logging` global lock may not be released appropriately, resulting in deadlock. -- components:

[issue39198] Lock may not be released in Logger.isEnabledFor

Change by Derek Brown : -- keywords: +patch pull_requests: +17236 stage: -> patch review pull_request: https://github.com/python/cpython/pull/17689 ___ Python tracker ___