From owner-freebsd-toolchain@FreeBSD.ORG Sun May 31 08:20:17 2015 Return-Path: Delivered-To: freebsd-toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 2F581754 for ; Sun, 31 May 2015 08:20:17 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 0FC4A1859 for ; Sun, 31 May 2015 08:20:17 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t4V8KGZo074125 for ; Sun, 31 May 2015 08:20:16 GMT (envelope-from daemon-user@phabric-backend.isc.freebsd.org) Received: (from daemon-user@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t4V8KG4I074124; Sun, 31 May 2015 08:20:16 GMT (envelope-from daemon-user) Date: Sun, 31 May 2015 08:20:16 +0000 To: freebsd-toolchain@freebsd.org From: "rodrigc (Craig Rodrigues)" Subject: [Differential] [Request, 22 lines] D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules Message-ID: X-Priority: 3 Thread-Topic: D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules X-Herald-Rules: <28> X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: Thread-Index: Y2M3ODgwNzhlNGU0M2Q5NjMyZTYwOGZmMTc0 Precedence: bulk X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: , , , MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="b1_8f01fff7f42147fd699240cbe1b1b6f3" X-BeenThere: freebsd-toolchain@freebsd.org X-Mailman-Version: 2.1.20 List-Id: Maintenance of FreeBSD's integrated toolchain List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 May 2015 08:20:17 -0000 --b1_8f01fff7f42147fd699240cbe1b1b6f3 Content-Type: text/plain; charset = "utf-8" Content-Transfer-Encoding: 8bit rodrigc created this revision. rodrigc added reviewers: araujo, bapt, emaste, dim. rodrigc added a subscriber: freebsd-toolchain. rodrigc set the repository for this revision to rS FreeBSD src repository. Herald added a subscriber: imp. REVISION SUMMARY Without this fix, compiling ypbind with gcc 4.9 will give warnings such as this: ``` /opt2/branches/head/usr.sbin/ypbind/ypbind.c: In function 'rpc_received': /opt2/branches/head/usr.sbin/ypbind/ypbind.c:948:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] *(u_int32_t *)&ybr.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_addr = raddrp->sin_addr.s_addr; ^ ``` REPOSITORY rS FreeBSD src repository REVISION DETAIL https://reviews.freebsd.org/D2690 AFFECTED FILES usr.sbin/ypbind/ypbind.c CHANGE DETAILS diff --git a/usr.sbin/ypbind/ypbind.c b/usr.sbin/ypbind/ypbind.c --- a/usr.sbin/ypbind/ypbind.c +++ b/usr.sbin/ypbind/ypbind.c @@ -225,10 +225,10 @@ res.ypbind_status = YPBIND_SUCC_VAL; res.ypbind_resp_u.ypbind_error = 0; /* Success */ - *(u_int32_t *)&res.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_addr = - ypdb->dom_server_addr.sin_addr.s_addr; - *(u_short *)&res.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_port = - ypdb->dom_server_addr.sin_port; + memcpy(&res.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_addr, + &ypdb->dom_server_addr.sin_addr.s_addr, sizeof(u_int32_t)); + memcpy(&res.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_port, + &ypdb->dom_server_addr.sin_port, sizeof(u_short)); /*printf("domain %s at %s/%d\n", ypdb->dom_domain, inet_ntoa(ypdb->dom_server_addr.sin_addr), ntohs(ypdb->dom_server_addr.sin_port));*/ @@ -275,8 +275,12 @@ bzero(&bindsin, sizeof bindsin); bindsin.sin_family = AF_INET; - bindsin.sin_addr.s_addr = *(u_int32_t *)argp->ypsetdom_binding.ypbind_binding_addr; - bindsin.sin_port = *(u_short *)argp->ypsetdom_binding.ypbind_binding_port; + memcpy(&bindsin.sin_addr.s_addr, + &argp->ypsetdom_binding.ypbind_binding_addr, + sizeof(u_int32_t)); + memcpy(&bindsin.sin_port, + &argp->ypsetdom_binding.ypbind_binding_port, + sizeof(u_short)); rpc_received(argp->ypsetdom_domain, &bindsin, 1); return((void *) &result); @@ -945,8 +949,10 @@ bzero(&ybr, sizeof ybr); ybr.ypbind_status = YPBIND_SUCC_VAL; - *(u_int32_t *)&ybr.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_addr = raddrp->sin_addr.s_addr; - *(u_short *)&ybr.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_port = raddrp->sin_port; + memcpy(ybr.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_addr, + &raddrp->sin_addr.s_addr, sizeof(u_int32_t)); + memcpy(&ybr.ypbind_resp_u.ypbind_bindinfo.ypbind_binding_port, + &raddrp->sin_port, sizeof(u_short)); if (writev(ypdb->dom_lockfd, iov, 2) != iov[0].iov_len + iov[1].iov_len) { syslog(LOG_WARNING, "write: %m"); EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: rodrigc, araujo, bapt, emaste, dim Cc: imp, freebsd-toolchain --b1_8f01fff7f42147fd699240cbe1b1b6f3 Content-Type: text/x-patch; charset=utf-8; name="D2690.5826.patch" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="D2690.5826.patch" ZGlmZiAtLWdpdCBhL3Vzci5zYmluL3lwYmluZC95cGJpbmQuYyBiL3Vzci5zYmluL3lwYmluZC95 cGJpbmQuYwotLS0gYS91c3Iuc2Jpbi95cGJpbmQveXBiaW5kLmMKKysrIGIvdXNyLnNiaW4veXBi aW5kL3lwYmluZC5jCkBAIC0yMjUsMTAgKzIyNSwxMCBAQAogCiAJcmVzLnlwYmluZF9zdGF0dXMg PSBZUEJJTkRfU1VDQ19WQUw7CiAJcmVzLnlwYmluZF9yZXNwX3UueXBiaW5kX2Vycm9yID0gMDsg LyogU3VjY2VzcyAqLwotCSoodV9pbnQzMl90ICopJnJlcy55cGJpbmRfcmVzcF91LnlwYmluZF9i aW5kaW5mby55cGJpbmRfYmluZGluZ19hZGRyID0KLQkJeXBkYi0+ZG9tX3NlcnZlcl9hZGRyLnNp bl9hZGRyLnNfYWRkcjsKLQkqKHVfc2hvcnQgKikmcmVzLnlwYmluZF9yZXNwX3UueXBiaW5kX2Jp bmRpbmZvLnlwYmluZF9iaW5kaW5nX3BvcnQgPQotCQl5cGRiLT5kb21fc2VydmVyX2FkZHIuc2lu X3BvcnQ7CisJbWVtY3B5KCZyZXMueXBiaW5kX3Jlc3BfdS55cGJpbmRfYmluZGluZm8ueXBiaW5k X2JpbmRpbmdfYWRkciwKKwkgICAgJnlwZGItPmRvbV9zZXJ2ZXJfYWRkci5zaW5fYWRkci5zX2Fk ZHIsIHNpemVvZih1X2ludDMyX3QpKTsKKwltZW1jcHkoJnJlcy55cGJpbmRfcmVzcF91LnlwYmlu ZF9iaW5kaW5mby55cGJpbmRfYmluZGluZ19wb3J0LAorCSAgICAmeXBkYi0+ZG9tX3NlcnZlcl9h ZGRyLnNpbl9wb3J0LCBzaXplb2YodV9zaG9ydCkpOwogCS8qcHJpbnRmKCJkb21haW4gJXMgYXQg JXMvJWRcbiIsIHlwZGItPmRvbV9kb21haW4sCiAJCWluZXRfbnRvYSh5cGRiLT5kb21fc2VydmVy X2FkZHIuc2luX2FkZHIpLAogCQludG9ocyh5cGRiLT5kb21fc2VydmVyX2FkZHIuc2luX3BvcnQp KTsqLwpAQCAtMjc1LDggKzI3NSwxMiBAQAogCiAJYnplcm8oJmJpbmRzaW4sIHNpemVvZiBiaW5k c2luKTsKIAliaW5kc2luLnNpbl9mYW1pbHkgPSBBRl9JTkVUOwotCWJpbmRzaW4uc2luX2FkZHIu c19hZGRyID0gKih1X2ludDMyX3QgKilhcmdwLT55cHNldGRvbV9iaW5kaW5nLnlwYmluZF9iaW5k aW5nX2FkZHI7Ci0JYmluZHNpbi5zaW5fcG9ydCA9ICoodV9zaG9ydCAqKWFyZ3AtPnlwc2V0ZG9t X2JpbmRpbmcueXBiaW5kX2JpbmRpbmdfcG9ydDsKKwltZW1jcHkoJmJpbmRzaW4uc2luX2FkZHIu c19hZGRyLAorCSAgICAmYXJncC0+eXBzZXRkb21fYmluZGluZy55cGJpbmRfYmluZGluZ19hZGRy LAorCSAgICBzaXplb2YodV9pbnQzMl90KSk7CisJbWVtY3B5KCZiaW5kc2luLnNpbl9wb3J0LAor CSAgICAmYXJncC0+eXBzZXRkb21fYmluZGluZy55cGJpbmRfYmluZGluZ19wb3J0LAorCSAgICBz aXplb2YodV9zaG9ydCkpOwogCXJwY19yZWNlaXZlZChhcmdwLT55cHNldGRvbV9kb21haW4sICZi aW5kc2luLCAxKTsKIAogCXJldHVybigodm9pZCAqKSAmcmVzdWx0KTsKQEAgLTk0NSw4ICs5NDks MTAgQEAKIAogCWJ6ZXJvKCZ5YnIsIHNpemVvZiB5YnIpOwogCXlici55cGJpbmRfc3RhdHVzID0g WVBCSU5EX1NVQ0NfVkFMOwotCSoodV9pbnQzMl90ICopJnlici55cGJpbmRfcmVzcF91LnlwYmlu ZF9iaW5kaW5mby55cGJpbmRfYmluZGluZ19hZGRyID0gcmFkZHJwLT5zaW5fYWRkci5zX2FkZHI7 Ci0JKih1X3Nob3J0ICopJnlici55cGJpbmRfcmVzcF91LnlwYmluZF9iaW5kaW5mby55cGJpbmRf YmluZGluZ19wb3J0ID0gcmFkZHJwLT5zaW5fcG9ydDsKKwltZW1jcHkoeWJyLnlwYmluZF9yZXNw X3UueXBiaW5kX2JpbmRpbmZvLnlwYmluZF9iaW5kaW5nX2FkZHIsCisJICAgICZyYWRkcnAtPnNp bl9hZGRyLnNfYWRkciwgc2l6ZW9mKHVfaW50MzJfdCkpOworCW1lbWNweSgmeWJyLnlwYmluZF9y ZXNwX3UueXBiaW5kX2JpbmRpbmZvLnlwYmluZF9iaW5kaW5nX3BvcnQsCisJICAgICZyYWRkcnAt PnNpbl9wb3J0LCBzaXplb2YodV9zaG9ydCkpOwogCiAJaWYgKHdyaXRldih5cGRiLT5kb21fbG9j a2ZkLCBpb3YsIDIpICE9IGlvdlswXS5pb3ZfbGVuICsgaW92WzFdLmlvdl9sZW4pIHsKIAkJc3lz bG9nKExPR19XQVJOSU5HLCAid3JpdGU6ICVtIik7Cgo= --b1_8f01fff7f42147fd699240cbe1b1b6f3-- From owner-freebsd-toolchain@FreeBSD.ORG Mon Jun 1 17:49:55 2015 Return-Path: Delivered-To: freebsd-toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 2F524236 for ; Mon, 1 Jun 2015 17:49:55 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 1272C1661 for ; Mon, 1 Jun 2015 17:49:55 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t51HnsVW090695 for ; Mon, 1 Jun 2015 17:49:54 GMT (envelope-from daemon-user@phabric-backend.isc.freebsd.org) Received: (from daemon-user@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t51HnsTt090694; Mon, 1 Jun 2015 17:49:54 GMT (envelope-from daemon-user) Date: Mon, 1 Jun 2015 17:49:54 +0000 To: freebsd-toolchain@freebsd.org From: "dim (Dimitry Andric)" Subject: [Differential] [Commented On] D1932: Remove the non-standard CC alias for c++ Message-ID: <3bb054cf8d22fd3ad23a8a4a71a79267@localhost.localdomain> X-Priority: 3 Thread-Topic: D1932: Remove the non-standard CC alias for c++ X-Herald-Rules: none, <28> X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: X-Phabricator-Cc: X-Phabricator-Cc: In-Reply-To: References: Thread-Index: NzZlYWQyYjJkNzMyMjZjMjM2MDkwMzY1NmZiIFVsm0I= Precedence: bulk X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" X-BeenThere: freebsd-toolchain@freebsd.org X-Mailman-Version: 2.1.20 List-Id: Maintenance of FreeBSD's integrated toolchain List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Jun 2015 17:49:55 -0000 dim added a comment. @imp, so are your concerns now addressed? I didn't expect many ports to fail, and there were just a handful, three of which were actual errors (they indended to run ${CC}, but somebody typo'd CC instead). REPOSITORY rS FreeBSD src repository REVISION DETAIL https://reviews.freebsd.org/D1932 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: dim, emaste, theraven, imp, bdrewery Cc: bdrewery, kib, imp, freebsd-toolchain From owner-freebsd-toolchain@FreeBSD.ORG Tue Jun 2 10:18:37 2015 Return-Path: Delivered-To: freebsd-toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id F1DA63EA for ; Tue, 2 Jun 2015 10:18:37 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id D33D816F3 for ; Tue, 2 Jun 2015 10:18:37 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t52AIblA096505 for ; Tue, 2 Jun 2015 10:18:37 GMT (envelope-from daemon-user@phabric-backend.isc.freebsd.org) Received: (from daemon-user@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t52AIbb0096504; Tue, 2 Jun 2015 10:18:37 GMT (envelope-from daemon-user) Date: Tue, 2 Jun 2015 10:18:37 +0000 To: freebsd-toolchain@freebsd.org From: "araujo (Marcelo Araujo)" Subject: [Differential] [Updated] D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules Message-ID: X-Priority: 3 Thread-Topic: D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules X-Herald-Rules: <28> X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: In-Reply-To: References: Thread-Index: Y2M3ODgwNzhlNGU0M2Q5NjMyZTYwOGZmMTc0IFVtgv0= Precedence: bulk X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: , MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" X-BeenThere: freebsd-toolchain@freebsd.org X-Mailman-Version: 2.1.20 List-Id: Maintenance of FreeBSD's integrated toolchain List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Jun 2015 10:18:38 -0000 araujo added a comment. Hi Craig, Could you take a look on this patch too? https://people.freebsd.org/~araujo/ypbind.patch I already integrated your changes from this revision. Maybe we should consider to sync it with the current OpenBSD version, although I'm not familiar with ypbind, maybe it is possible. http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/ypbind/ypbind.c?annotate=1.63 Best, REPOSITORY rS FreeBSD src repository REVISION DETAIL https://reviews.freebsd.org/D2690 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: rodrigc, bapt, emaste, dim, araujo Cc: imp, freebsd-toolchain From owner-freebsd-toolchain@FreeBSD.ORG Wed Jun 3 06:42:26 2015 Return-Path: Delivered-To: freebsd-toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 0B4074B7 for ; Wed, 3 Jun 2015 06:42:26 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id E290C1458 for ; Wed, 3 Jun 2015 06:42:25 +0000 (UTC) (envelope-from daemon-user@FreeBSD.org) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t536gPdh027817 for ; Wed, 3 Jun 2015 06:42:25 GMT (envelope-from daemon-user@phabric-backend.isc.freebsd.org) Received: (from daemon-user@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t536gP4Z027816; Wed, 3 Jun 2015 06:42:25 GMT (envelope-from daemon-user) Date: Wed, 3 Jun 2015 06:42:25 +0000 To: freebsd-toolchain@freebsd.org From: "dim (Dimitry Andric)" Subject: [Differential] [Commented On] D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules Message-ID: X-Priority: 3 Thread-Topic: D2690: In ypbind, eliminate error: dereferencing type-punned pointer will break strict-aliasing rules X-Herald-Rules: <28> X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: In-Reply-To: References: Thread-Index: Y2M3ODgwNzhlNGU0M2Q5NjMyZTYwOGZmMTc0IFVuodE= Precedence: bulk X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" X-BeenThere: freebsd-toolchain@freebsd.org X-Mailman-Version: 2.1.20 List-Id: Maintenance of FreeBSD's integrated toolchain List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Jun 2015 06:42:26 -0000 dim added inline comments. INLINE COMMENTS usr.sbin/ypbind/ypbind.c:952 Missing an & here? REPOSITORY rS FreeBSD src repository REVISION DETAIL https://reviews.freebsd.org/D2690 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: rodrigc, bapt, emaste, dim, araujo Cc: imp, freebsd-toolchain