fix usage of 'memo' in '__deepcopy__' implementation

Author: herin049

opentelemetry-api/src/opentelemetry/attributes/__init__.py

@@ -320,18 +320,16 @@ def __len__(self) -> int:
         return len(self._dict)
 
     def __deepcopy__(self, memo: dict) -> "BoundedAttributes":
-        with self._lock:
-            attributes = copy.deepcopy(self._dict, memo)
-            dropped = self.dropped
         copy_ = BoundedAttributes(
-            self.maxlen,
-            attributes,
-            self._immutable,
-            self.max_value_len,
-            self._extended_attributes,
+            maxlen=self.maxlen,
+            immutable=self._immutable,
+            max_value_len=self.max_value_len,
+            extended_attributes=self._extended_attributes,
         )
-        copy_.dropped = dropped
         memo[id(self)] = copy_
+        with self._lock:
+            copy_._dict = copy.deepcopy(self._dict, memo)
+            copy_.dropped = self.dropped
         return copy_
 
     def copy(self):  # type: ignore

opentelemetry-sdk/src/opentelemetry/sdk/util/__init__.py

@@ -58,6 +58,7 @@ def __init__(self, maxlen: Optional[int]):
 
     def __deepcopy__(self, memo):
         copy_ = BoundedList(0)
+        memo[id(self)] = copy_
         with self._lock:
             copy_.dropped = self.dropped
             copy_._dq = copy.deepcopy(self._dq, memo)