Author: pfg
Date: Sat Jul 14 20:24:00 2012
New Revision: 238457
URL: http://svn.freebsd.org/changeset/base/238457

Log:
  Merge illumos commit 13455:7205f7794835
  
  1458 D compiler fails to generate error on sizeof() an undefined struct
  
  Reference:
  https://www.illumos.org/issues/1458
  
  Tested by:    Fabian Keil
  MFC after:    3 weeks

Added:
  
head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d
     - copied unchanged from r238456, 
vendor/illumos/dist/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d
Modified:
  head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_parser.c

Copied: 
head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d
 (from r238456, 
vendor/illumos/dist/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d)
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ 
head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d
       Sat Jul 14 20:24:00 2012        (r238457, copy of r238456, 
vendor/illumos/dist/cmd/dtrace/test/tst/common/sizeof/err.D_SIZEOF_TYPE.badstruct.d)
@@ -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);
+}

Modified: head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_parser.c
==============================================================================
--- head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_parser.c      Sat Jul 
14 20:14:03 2012        (r238456)
+++ head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_parser.c      Sat Jul 
14 20:24:00 2012        (r238457)
@@ -22,7 +22,7 @@
 
 /*
  * Copyright 2006 Sun Microsystems, Inc.  All rights reserved.
- * Use is subject to license terms.
+ * Copyright (c) 2011, Joyent Inc. All rights reserved.
  */
 
 #pragma ident  "%Z%%M% %I%     %E% SMI"
@@ -725,12 +725,19 @@ dt_node_type_name(const dt_node_t *dnp, 
 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));
 }
 
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to