PR python/14635:

	* python/py-symtab.c (del_objfile_sal): Set 'symtab' field
	to Py_None.
testsuite
	* gdb.python/py-symtab.exp: Test sal and symtab destructors.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 5308770..5f5eaf9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,11 @@
 2012-10-15  Tom Tromey  <tromey@redhat.com>
 
+       PR python/14635:
+       * python/py-symtab.c (del_objfile_sal): Set 'symtab' field
+       to Py_None.
+
+2012-10-15  Tom Tromey  <tromey@redhat.com>
+
        PR python/14634:
        * python/py-symbol.c (sympy_dealloc): Check for NULL symbol.
 

diff --git a/gdb/python/py-symtab.c b/gdb/python/py-symtab.c
index b6f45e6..21a172c 100644 (file)
--- a/gdb/python/py-symtab.c
+++ b/gdb/python/py-symtab.c
@@ -470,7 +470,10 @@ del_objfile_sal (struct objfile *objfile, void *datum)
     {
       sal_object *next = obj->next;
 
-      obj->symtab = NULL;
+      Py_DECREF (obj->symtab);
+      obj->symtab = (symtab_object *) Py_None;
+      Py_INCREF (Py_None);
+
       obj->next = NULL;
       obj->prev = NULL;
       xfree (obj->sal);

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 71c2d85..38a3693 100644 (file)
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,9 @@
 2012-10-15  Tom Tromey  <tromey@redhat.com>
 
+       * gdb.python/py-symtab.exp: Test sal and symtab destructors.
+
+2012-10-15  Tom Tromey  <tromey@redhat.com>
+
        * gdb.python/py-symbol.exp: Test symbol destructor.
 
 2012-10-14  Yao Qi  <yao@codesourcery.com>

diff --git a/gdb/testsuite/gdb.python/py-symtab.exp b/gdb/testsuite/gdb.python/py-symtab.exp
index da0c3e4..0d3a5f2 100644 (file)
--- a/gdb/testsuite/gdb.python/py-symtab.exp
+++ b/gdb/testsuite/gdb.python/py-symtab.exp
@@ -73,3 +73,6 @@ gdb_test "python print \"simple_struct\" in static_symbols" "True" "Test simple_
 gdb_unload
 gdb_test "python print sal.is_valid()" "False" "Test sal.is_valid"
 gdb_test "python print symtab.is_valid()" "False" "Test symtab.is_valid()"
+
+gdb_test_no_output "python sal = None" "Test sal destructor"
+gdb_test_no_output "python symtab = None" "Test symtab destructor"