1458 D compiler fails to generate error on sizeof() an undefined struct
authorBryan Cantrill <bryan@joyent.com>
Thu, 07 Jul 2011 20:20:29 -0700
changeset 13455 7205f7794835
parent 13454 8f54cb1a45aa
child 13456 70af5b84f732
1458 D compiler fails to generate error on sizeof() an undefined struct Reviewed by: Gordon Ross <[email protected]> Reviewed by: Eric Schrock <[email protected]> Approved by: Garrett D'Amore <[email protected]>
usr/src/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d
usr/src/lib/libdtrace/common/dt_parser.c
usr/src/pkg/manifests/system-dtrace-tests.mf
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/usr/src/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d	Thu Jul 07 20:20:29 2011 -0700
@@ -0,0 +1,30 @@
+/*
+ * CDDL HEADER START
+ *
+ * The contents of this file are subject to the terms of the
+ * Common Development and Distribution License (the "License").
+ * You may not use this file except in compliance with the License.
+ *
+ * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
+ * or http://www.opensolaris.org/os/licensing.
+ * See the License for the specific language governing permissions
+ * and limitations under the License.
+ *
+ * When distributing Covered Code, include this CDDL HEADER in each
+ * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
+ * If applicable, add the following below this CDDL HEADER, with the
+ * fields enclosed by brackets "[]" replaced with your own identifying
+ * information: Portions Copyright [yyyy] [name of copyright owner]
+ *
+ * CDDL HEADER END
+ */
+
+/*
+ * Copyright (c) 2011, Joyent Inc. All rights reserved.
+ */
+
+BEGIN
+{
+	trace(sizeof (struct suckitlarry));
+	exit(0);
+}
--- a/usr/src/lib/libdtrace/common/dt_parser.c	Wed Jul 06 00:27:16 2011 -0700
+++ b/usr/src/lib/libdtrace/common/dt_parser.c	Thu Jul 07 20:20:29 2011 -0700
@@ -21,6 +21,7 @@
 
 /*
  * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, Joyent Inc. All rights reserved.
  */
 
 /*
@@ -719,12 +720,19 @@
 size_t
 dt_node_type_size(const dt_node_t *dnp)
 {
+	ctf_id_t base;
+
 	if (dnp->dn_kind == DT_NODE_STRING)
 		return (strlen(dnp->dn_string) + 1);
 
 	if (dt_node_is_dynamic(dnp) && dnp->dn_ident != NULL)
 		return (dt_ident_size(dnp->dn_ident));
 
+	base = ctf_type_resolve(dnp->dn_ctfp, dnp->dn_type);
+
+	if (ctf_type_kind(dnp->dn_ctfp, base) == CTF_K_FORWARD)
+		return (0);
+
 	return (ctf_type_size(dnp->dn_ctfp, dnp->dn_type));
 }
 
--- a/usr/src/pkg/manifests/system-dtrace-tests.mf	Wed Jul 06 00:27:16 2011 -0700
+++ b/usr/src/pkg/manifests/system-dtrace-tests.mf	Thu Jul 07 20:20:29 2011 -0700
@@ -1532,6 +1532,8 @@
     mode=0444
 file path=opt/SUNWdtrt/tst/common/sizeof/err.D_IDENT_UNDEF.UnknownSymbol.d \
     mode=0444
+file path=opt/SUNWdtrt/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d \
+    mode=0444
 file path=opt/SUNWdtrt/tst/common/sizeof/err.D_SIZEOF_TYPE.d mode=0444
 file path=opt/SUNWdtrt/tst/common/sizeof/err.D_SYNTAX.SizeofBadType.d \
     mode=0444