Fix PR c++/68711 - [6 regression] SEGV on an invalid offsetof of a member

of a virtual base.

gcc/testsuite/ChangeLog:
	* g++.dg/other/offsetof8.C: New test.

gcc/cp/ChangeLog:
	* typeck.c (build_class_member_access_expr): Strip NOPs before
        testing a potentially null operand for equality to zero.

From-SVN: r231437

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 7569a625a7c315978df63ec253bc36c96657712f..ac41ca655c5956c9f0c5de0abad23e6b01ad44ed 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2015-12-08  Martin Sebor  <msebor@redhat.com>
+
+       PR c++/68711
+       * typeck.c (build_class_member_access_expr): Strip NOPs before
+        testing a potentially null operand for equality to zero.
+
 2015-12-07  Jakub Jelinek  <jakub@redhat.com>
 
        PR c++/68760

diff --git a/gcc/cp/typeck.c b/gcc/cp/typeck.c
index 3cf3c953f4b8d96efd09d6023e3b3eaf0843a65b..17671ee838d97d22030351a1d99f5c88a4c28523 100644 (file)
--- a/gcc/cp/typeck.c
+++ b/gcc/cp/typeck.c
@@ -2358,8 +2358,11 @@ build_class_member_access_expr (cp_expr object, tree member,
       int type_quals;
       tree member_type;
 
-      null_object_p = (INDIRECT_REF_P (object)
-                      && integer_zerop (TREE_OPERAND (object, 0)));
+      if (INDIRECT_REF_P (object))
+       null_object_p =
+         integer_zerop (tree_strip_nop_conversions (TREE_OPERAND (object, 0)));
+      else
+       null_object_p = false;
 
       /* Convert OBJECT to the type of MEMBER.  */
       if (!same_type_p (TYPE_MAIN_VARIANT (object_type),

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index b596b0338470a641c0e0bf9e8d42ec1b33dbbe39..5cefc07641dec027c76caf08081daf0ca11e40df 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2015-12-08  Martin Sebor  <msebor@redhat.com>
+
+       PR c++/68711
+       * g++.dg/other/offsetof8.C: New test.
+
 2015-12-08  Nathan Sidwell  <nathan@acm.org>
 
        * gcc.target/nvptx/trailing-init.c: New.

diff --git a/gcc/testsuite/g++.dg/other/offsetof8.C b/gcc/testsuite/g++.dg/other/offsetof8.C
new file mode 100644 (file)
index 0000000..daca70a
--- /dev/null
+++ b/gcc/testsuite/g++.dg/other/offsetof8.C
@@ -0,0 +1,12 @@
+// PR c++/68711 - [5 regression] SEGV on an invalid offsetof of a member
+//                of a virtual base
+// { dg-do compile }
+
+struct A { int i; };
+
+struct B: virtual A { };
+
+int a[]  = {
+  !&((B*)0)->i,    // { dg-error "invalid access to non-static data member" }
+  __builtin_offsetof (B, i)   // { dg-error "invalid access to non-static" }
+};