From: Luis Machado <luisgpm@br.ibm.com>
Date: Thu, 8 Mar 2012 21:07:25 +0000 (+0000)
Subject: 2012-03-08  Luis Machado  <lgustavo@codesourcery.com>
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=52323be946ffc98caecb4c76ec0801e424b913ae;p=deliverable%2Fbinutils-gdb.git

2012-03-08  Luis Machado  <lgustavo@codesourcery.com>

	* ax-gdb.c (gen_fetch): Fail gracefully and use error instead
	of internal error for unknown/unsupported types.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9eb949bc17..611c900e0e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2012-03-08  Luis Machado  <lgustavo@codesourcery.com>
+
+	* ax-gdb.c (gen_fetch): Fail gracefully and use error instead
+	of internal error for unknown/unsupported types.
+
 2012-03-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
 	Fix CU relative vs. absolute DIE offsets.
diff --git a/gdb/ax-gdb.c b/gdb/ax-gdb.c
index 37882be0a5..cc057f89a0 100644
--- a/gdb/ax-gdb.c
+++ b/gdb/ax-gdb.c
@@ -553,12 +553,11 @@ gen_fetch (struct agent_expr *ax, struct type *type)
       break;
 
     default:
-      /* Either our caller shouldn't have asked us to dereference that
-         pointer (other code's fault), or we're not implementing
-         something we should be (this code's fault).  In any case,
-         it's a bug the user shouldn't see.  */
-      internal_error (__FILE__, __LINE__,
-		      _("gen_fetch: bad type code"));
+      /* Our caller requested us to dereference a pointer from an unsupported
+	 type.  Error out and give callers a chance to handle the failure
+	 gracefully.  */
+      error (_("gen_fetch: Unsupported type code `%s'."),
+	     TYPE_NAME (type));
     }
 }