[RFA] dwarf2loc.c, loclist_read_variable, Assert frame not null

[Michael Snyder <msnyder@vmware.com>]

[-- Attachment #1: Type: text/plain, Size: 162 bytes --]

Now, I've heard a rumor that frame-infos can never be null.

If that's true, then instead of this patch, we should just
delete the earlier check for null.

OK?



[-- Attachment #2: null5.txt --]
[-- Type: text/plain, Size: 790 bytes --]

2011-03-04  Michael Snyder  <msnyder@vmware.com>

	* dwarf2loc.c (loclist_read_variable): Assert frame not null.

Index: dwarf2loc.c
===================================================================
RCS file: /cvs/src/src/gdb/dwarf2loc.c,v
retrieving revision 1.110
diff -u -p -u -p -r1.110 dwarf2loc.c
--- dwarf2loc.c	27 Feb 2011 00:01:12 -0000	1.110
+++ dwarf2loc.c	4 Mar 2011 22:34:24 -0000
@@ -2687,8 +2687,11 @@ loclist_read_variable (struct symbol *sy
       set_value_optimized_out (val, 1);
     }
   else
-    val = dwarf2_evaluate_loc_desc (SYMBOL_TYPE (symbol), frame, data, size,
-				    dlbaton->per_cu);
+    {
+      gdb_asert (frame);
+      val = dwarf2_evaluate_loc_desc (SYMBOL_TYPE (symbol), frame,
+				      data, size, dlbaton->per_cu);
+    }
 
   return val;
 }