This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Exception and contextmanager in __getattr__ causes reference cycle
Type: behavior Stage:
Components: Library (Lib) Versions: Python 3.6
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: crccw
Priority: normal Keywords:

Created on 2021-03-18 00:49 by crccw, last changed 2022-04-11 14:59 by admin.

Messages (1)
msg388977 - (view) Author: Ran Chen (crccw) Date: 2021-03-18 00:49
If __getattr__ raises exception within a contextlib.context_manager, it creates a reference cycle and prevents the frame from being garbage collected. This only happens if the exception is raised inside a context manager inside __getattr__. It doesn't happen if there's no context manager.

Repro:

```
import contextlib
import gc

@contextlib.contextmanager
def ct():
  yield


class A(object):

  def __getattr__(self, name):
    with ct():
      raise AttributeError()
  


def f():
  a = A()
  hasattr(a, 'notexist')


gc.set_debug(gc.DEBUG_LEAK)
f()
gc.collect()
```

It also doesn't happen if we catch the exception outside of the context manager and re-raise it:

```
  def __getattr__(self, name):
    try:
      with ct():
        raise AttributeError()
    except:
      raise
```
History
Date User Action Args
2022-04-11 14:59:42adminsetgithub: 87699
2021-03-18 00:49:33crccwcreate