From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout4.zoneedit.com (mailout4.zoneedit.com [64.68.198.64]) by mail.openembedded.org (Postfix) with ESMTP id 4B8027C420 for ; Tue, 5 Mar 2019 01:12:09 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by mailout4.zoneedit.com (Postfix) with ESMTP id 4E83740B0A; Tue, 5 Mar 2019 01:12:10 +0000 (UTC) Received: from mailout4.zoneedit.com ([127.0.0.1]) by localhost (zmo14-pco.easydns.vpn [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xeS5rH7qXnJz; Tue, 5 Mar 2019 01:12:10 +0000 (UTC) Received: from mail.denix.org (pool-100-15-91-218.washdc.fios.verizon.net [100.15.91.218]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mailout4.zoneedit.com (Postfix) with ESMTPSA id 2BCFA40A58; Tue, 5 Mar 2019 01:12:09 +0000 (UTC) Received: by mail.denix.org (Postfix, from userid 1000) id D9D7016BD48; Mon, 4 Mar 2019 20:12:08 -0500 (EST) Date: Mon, 4 Mar 2019 20:12:08 -0500 From: Denys Dmytriyenko To: openembedded-core@lists.openembedded.org Message-ID: <20190305011208.GC20517@denix.org> References: <1551723073-3036-1-git-send-email-denis@denix.org> MIME-Version: 1.0 In-Reply-To: <1551723073-3036-1-git-send-email-denis@denix.org> User-Agent: Mutt/1.5.20 (2009-06-14) Subject: Re: [thud][PATCH] openssl10: Fix mutliple include assumptions for des.h in opensslconf.h X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Mar 2019 01:12:09 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Armin, Not sure when would be the next batch of thud backports - as discussed earlier, I needed this fix to un-break the last set of backported updates from last Monday. Please let me know when this can be merged to thud. Thanks! -- Denys On Mon, Mar 04, 2019 at 01:11:13PM -0500, Denys Dmytriyenko wrote: > From: Denys Dmytriyenko > > The fix is heavily based on Khem's previous fix for bn.h/BN_LLONG breakage: > https://git.openembedded.org/openembedded-core/commit/?id=f787b0bb9b0626ddbf2ac94cb206c76716a3773d > > (From OE-Core rev: 914e1520bf9c45e14bce9993c9131a2c0702b9c9) > > Signed-off-by: Denys Dmytriyenko > Cc: Khem Raj > Signed-off-by: Richard Purdie > Signed-off-by: Denys Dmytriyenko > --- > .../openssl10/0001-Fix-DES_LONG-breakage.patch | 35 ++++++++++++++++++++++ > .../openssl/openssl10_1.0.2q.bb | 1 + > 2 files changed, 36 insertions(+) > create mode 100644 meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch > > diff --git a/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch > new file mode 100644 > index 0000000..7243fb4 > --- /dev/null > +++ b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch > @@ -0,0 +1,35 @@ > +From 1aec49cc45e7cf5ebc059a77081ac3ea2a5aff7b Mon Sep 17 00:00:00 2001 > +From: Denys Dmytriyenko > +Date: Fri, 1 Mar 2019 13:14:56 -0500 > +Subject: [PATCH] Fix DES_LONG breakage > + > +Mimic previous BN_LLONG fix by Khem Raj here. Re-use its description: > + > +opensslconf.h is defining DES_LONG only when included from des.h which > +is not robust at all, especially when include guards are used and > +multiple inclusions of a given header is not allowed. so lets take out > +the nesting constraint and add OPENSSL_SYS_UEFI constraint instead > + > +Upstream-Status: Inappropriate [ fixed differently with OpenSSL 1.1+ ] > + > +Signed-off-by: Denys Dmytriyenko > +--- > + crypto/opensslconf.h.in | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/crypto/opensslconf.h.in b/crypto/opensslconf.h.in > +index a10c10f..0147a4d 100644 > +--- a/crypto/opensslconf.h.in > ++++ b/crypto/opensslconf.h.in > +@@ -48,7 +48,7 @@ > + #endif > + #endif > + > +-#if (defined(HEADER_NEW_DES_H) || defined(HEADER_DES_H)) && !defined(DES_LONG) > ++#if !defined(OPENSSL_SYS_UEFI) && !defined(DES_LONG) > + /* If this is set to 'unsigned int' on a DEC Alpha, this gives about a > + * %20 speed up (longs are 8 bytes, int's are 4). */ > + #ifndef DES_LONG > +-- > +2.7.4 > + > diff --git a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb > index 355dbdc..54af100 100644 > --- a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb > +++ b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb > @@ -41,6 +41,7 @@ SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz \ > file://0001-openssl-force-soft-link-to-avoid-rare-race.patch \ > file://0001-allow-manpages-to-be-disabled.patch \ > file://0001-Fix-BN_LLONG-breakage.patch \ > + file://0001-Fix-DES_LONG-breakage.patch \ > " > > SRC_URI_append_class-target = " \ > -- > 2.7.4 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core