2009-07-07 Paul Pluzhnikov <ppluzhnikov@google.com>

	* python/python-value.c (valpy_getitem): Don't return from TRY_CATCH.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 56480f10298cea9792cae758eee742054af798d3..1178a93ddf789a06e8e8492cebf1399f2ab3bb1d 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2009-07-07  Paul Pluzhnikov  <ppluzhnikov@google.com>
+
+       * python/python-value.c (valpy_getitem): Don't return from TRY_CATCH.
+
 2009-07-07  Sami Wagiaalla  <swagiaal@redhat.com>
 
        * cp-support.h (struct using_direct): Rename members inner and

diff --git a/gdb/python/python-value.c b/gdb/python/python-value.c
index 5304cafa2a6c561881db255edb3dc754f2ee8766..948ff06d366859e80d1b56ee98f465c18eb0327a 100644 (file)
--- a/gdb/python/python-value.c
+++ b/gdb/python/python-value.c
@@ -267,8 +267,7 @@ valpy_getitem (PyObject *self, PyObject *key)
 {
   value_object *self_value = (value_object *) self;
   char *field = NULL;
-  struct value *idx = NULL;
-  struct value *res_val = NULL;          /* Initialize to appease gcc warning.  */
+  struct value *res_val = NULL;
   volatile struct gdb_exception except;
 
   if (gdbpy_is_string (key))
@@ -290,12 +289,17 @@ valpy_getitem (PyObject *self, PyObject *key)
             value code throw an exception if the index has an invalid
             type.  */
          struct value *idx = convert_value_from_python (key);
-         if (idx == NULL)
-           return NULL;
-
-         res_val = value_subscript (tmp, value_as_long (idx));
+         if (idx != NULL)
+           res_val = value_subscript (tmp, value_as_long (idx));
        }
     }
+
+  if (res_val == NULL)
+    {
+      gdb_assert (field == NULL);
+      return NULL;
+    }
+
   if (field)
     xfree (field);
   GDB_PY_HANDLE_EXCEPTION (except);