From 89752fb6d465c78026befcc5a61e8af655587ad7 Mon Sep 17 00:00:00 2001
From: Julian Andres Klode <jak@debian.org>
Date: Sat, 23 Jan 2010 19:24:04 +0100
Subject: python/acquire-item.cc: Support items without an owner set.

---
 python/acquire-item.cc | 38 ++++++++++++++++++++------------------
 1 file changed, 20 insertions(+), 18 deletions(-)

(limited to 'python')

diff --git a/python/acquire-item.cc b/python/acquire-item.cc
index 1fb66080..73aec1d6 100644
--- a/python/acquire-item.cc
+++ b/python/acquire-item.cc
@@ -178,26 +178,28 @@ static void acquireitem_dealloc(PyObject *self)
 {
     pkgAcquire::Item *item = PyAcquireItem_ToCpp(self);
     PyAcquireObject *Owner = (PyAcquireObject *)GetOwner<pkgAcquire::Item*>(self);
-    PyAcquireItems item_struct = Owner->items[item];
-    // TODO: Unregister the object in the owner.
-    if (!((CppOwnedPyObject<pkgAcquire::Item*>*)self)->NoDelete) {
-        if (item_struct.file != 0 && item_struct.file != self)
-            item_struct.file->Object = 0;
-        if (item_struct.item != 0 && item_struct.item != self) {
-            item_struct.item->Object = 0;
-            Py_DECREF(item_struct.item);
+    if (Owner != NULL) {
+        PyAcquireItems item_struct = Owner->items[item];
+        // TODO: Unregister the object in the owner.
+        if (!((CppOwnedPyObject<pkgAcquire::Item*>*)self)->NoDelete) {
+            if (item_struct.file != 0 && item_struct.file != self)
+                item_struct.file->Object = 0;
+            if (item_struct.item != 0 && item_struct.item != self) {
+                item_struct.item->Object = 0;
+                Py_DECREF(item_struct.item);
+            }
+            if (item_struct.desc != 0) {
+                item_struct.desc->Object = 0;
+                Py_DECREF(item_struct.desc);
+            }
+            Owner->items.erase(item);
         }
-        if (item_struct.desc != 0) {
-            item_struct.desc->Object = 0;
-            Py_DECREF(item_struct.desc);
+        else {
+            if (item_struct.file == self)
+                item_struct.file = 0;
+            if (item_struct.item == self)
+                item_struct.item = 0;
         }
-        Owner->items.erase(item);
-    }
-    else {
-        if (item_struct.file == self)
-            item_struct.file = 0;
-        if (item_struct.item == self)
-            item_struct.item = 0;
     }
 
     CppOwnedDeallocPtr<pkgAcquire::Item*>(self);
-- 
cgit v1.2.3