Patchwork Fix PR target/48807

login
register
mail settings
Submitter Henderson, Stuart
Date May 6, 2011, 10:50 a.m.
Message ID <05E9E85E39C35B4D96ED3A3190E35A10A49CC3C61F@LIMKCMBX1.ad.analog.com>
Download mbox | patch
Permalink /patch/94360/
State New
Headers show

Comments

Henderson, Stuart - May 6, 2011, 10:50 a.m.
Hi,
The attached patch is a fix for PR/48807, which is a segfault when compiling the bfin compiler.  The problem appears to be that we're not checking the return value of cgraph_local_info for null before attempting to dereference it.  This wasn't a problem before, but now cgraph_local_info calls cgraph_get_node (instead of the old cgraph_node), we cannot assume it will always return non-null.

Fix is in bfin specific code.

Ok to commit to trunk?

Stu


2011-05-06  Stuart Henderson  <shenders@gcc.gnu.org>

    * config/bfin/bfin.c: Check return value of cgraph_local_info for null before
        attempting to use it.

Patch

Index: gcc/config/bfin/bfin.c
===================================================================
--- gcc/config/bfin/bfin.c      (revision 173363)
+++ gcc/config/bfin/bfin.c      (working copy)
@@ -2077,6 +2077,8 @@ 

   this_func = cgraph_local_info (current_function_decl);
   called_func = cgraph_local_info (decl);
+  if (!called_func || !this_func)
+    return false;
   return !called_func->local || this_func->local;
 }