From nobody Thu May 2 19:50:30 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Authentication-Results: mx.zoho.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 149145822083722.49996789967213; Wed, 5 Apr 2017 22:57:00 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 88ACE21942337; Wed, 5 Apr 2017 22:56:59 -0700 (PDT) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id B951A2194232A for ; Wed, 5 Apr 2017 22:56:58 -0700 (PDT) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Apr 2017 22:56:58 -0700 Received: from shwde6388.ccr.corp.intel.com ([10.239.9.17]) by fmsmga001.fm.intel.com with ESMTP; 05 Apr 2017 22:56:56 -0700 X-Original-To: edk2-devel@lists.01.org DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1491458218; x=1522994218; h=from:to:cc:subject:date:message-id; bh=w3+6kkkZKEF36EQDGASTa7QyrjTaqV/BVAfncxJ9h8U=; b=PneWH8HXY9+JeHwWl1N1+e9TUryTbyoI3CUokMOWZY1+UFLfQDRY2gSC LUtH5znsMeCvQ4OrTzrtW5fPirB2FQ==; X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,282,1488873600"; d="scan'208";a="1131669710" From: Long Qin To: liming.gao@intel.com, ting.ye@intel.com Date: Thu, 6 Apr 2017 13:56:12 +0800 Message-Id: <20170406055612.10168-1-qin.long@intel.com> X-Mailer: git-send-email 2.12.2.windows.1 Subject: [edk2] [Patch] CryptoPkg: Move openssl and CRT headers to private include section X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: edk2-devel@lists.01.org, Qin Long MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Moving the header files for openssl and CRT wrappers to the private include section, since these files should be referenced by CryptoPkg internally. This update was supported by new [Includes.Common.Private] setting in Package DEC file. The external consumer modules should only use the interfaces defined in BaseCryptLib.h to access crypto functions. This change will be helpful to immediately detect any illegal direct reference to internal openssl headers. The Perl script "process_files.pl" was also updated to reflect the new private include path. Cc: Gao Liming Cc: Ting Ye Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qin Long Signed-off-by: Long Qin --- CryptoPkg/CryptoPkg.dec | 3 +++ CryptoPkg/{ =3D> Library}/Include/CrtLibSupport.h | 0 CryptoPkg/{ =3D> Library}/Include/assert.h | 0 CryptoPkg/{ =3D> Library}/Include/ctype.h | 0 CryptoPkg/{ =3D> Library}/Include/errno.h | 0 CryptoPkg/{ =3D> Library}/Include/internal/dso_conf.h | 0 CryptoPkg/{ =3D> Library}/Include/limits.h | 0 CryptoPkg/{ =3D> Library}/Include/memory.h | 0 CryptoPkg/{ =3D> Library}/Include/openssl/opensslconf.h | 0 CryptoPkg/{ =3D> Library}/Include/stdarg.h | 0 CryptoPkg/{ =3D> Library}/Include/stddef.h | 0 CryptoPkg/{ =3D> Library}/Include/stdio.h | 0 CryptoPkg/{ =3D> Library}/Include/stdlib.h | 0 CryptoPkg/{ =3D> Library}/Include/string.h | 0 CryptoPkg/{ =3D> Library}/Include/strings.h | 0 CryptoPkg/{ =3D> Library}/Include/sys/time.h | 0 CryptoPkg/{ =3D> Library}/Include/sys/types.h | 0 CryptoPkg/{ =3D> Library}/Include/syslog.h | 0 CryptoPkg/{ =3D> Library}/Include/time.h | 0 CryptoPkg/{ =3D> Library}/Include/unistd.h | 0 CryptoPkg/Library/OpensslLib/process_files.pl | 2 +- 21 files changed, 4 insertions(+), 1 deletion(-) rename CryptoPkg/{ =3D> Library}/Include/CrtLibSupport.h (100%) rename CryptoPkg/{ =3D> Library}/Include/assert.h (100%) rename CryptoPkg/{ =3D> Library}/Include/ctype.h (100%) rename CryptoPkg/{ =3D> Library}/Include/errno.h (100%) rename CryptoPkg/{ =3D> Library}/Include/internal/dso_conf.h (100%) rename CryptoPkg/{ =3D> Library}/Include/limits.h (100%) rename CryptoPkg/{ =3D> Library}/Include/memory.h (100%) rename CryptoPkg/{ =3D> Library}/Include/openssl/opensslconf.h (100%) rename CryptoPkg/{ =3D> Library}/Include/stdarg.h (100%) rename CryptoPkg/{ =3D> Library}/Include/stddef.h (100%) rename CryptoPkg/{ =3D> Library}/Include/stdio.h (100%) rename CryptoPkg/{ =3D> Library}/Include/stdlib.h (100%) rename CryptoPkg/{ =3D> Library}/Include/string.h (100%) rename CryptoPkg/{ =3D> Library}/Include/strings.h (100%) rename CryptoPkg/{ =3D> Library}/Include/sys/time.h (100%) rename CryptoPkg/{ =3D> Library}/Include/sys/types.h (100%) rename CryptoPkg/{ =3D> Library}/Include/syslog.h (100%) rename CryptoPkg/{ =3D> Library}/Include/time.h (100%) rename CryptoPkg/{ =3D> Library}/Include/unistd.h (100%) diff --git a/CryptoPkg/CryptoPkg.dec b/CryptoPkg/CryptoPkg.dec index fdccbf06f7..b2fae6142a 100644 --- a/CryptoPkg/CryptoPkg.dec +++ b/CryptoPkg/CryptoPkg.dec @@ -24,6 +24,9 @@ =20 [Includes] Include + +[Includes.Common.Private] + Library/Include Library/OpensslLib/openssl/include Library/OpensslLib/openssl/crypto/include =20 diff --git a/CryptoPkg/Include/CrtLibSupport.h b/CryptoPkg/Library/Include/= CrtLibSupport.h similarity index 100% rename from CryptoPkg/Include/CrtLibSupport.h rename to CryptoPkg/Library/Include/CrtLibSupport.h diff --git a/CryptoPkg/Include/assert.h b/CryptoPkg/Library/Include/assert.h similarity index 100% rename from CryptoPkg/Include/assert.h rename to CryptoPkg/Library/Include/assert.h diff --git a/CryptoPkg/Include/ctype.h b/CryptoPkg/Library/Include/ctype.h similarity index 100% rename from CryptoPkg/Include/ctype.h rename to CryptoPkg/Library/Include/ctype.h diff --git a/CryptoPkg/Include/errno.h b/CryptoPkg/Library/Include/errno.h similarity index 100% rename from CryptoPkg/Include/errno.h rename to CryptoPkg/Library/Include/errno.h diff --git a/CryptoPkg/Include/internal/dso_conf.h b/CryptoPkg/Library/Incl= ude/internal/dso_conf.h similarity index 100% rename from CryptoPkg/Include/internal/dso_conf.h rename to CryptoPkg/Library/Include/internal/dso_conf.h diff --git a/CryptoPkg/Include/limits.h b/CryptoPkg/Library/Include/limits.h similarity index 100% rename from CryptoPkg/Include/limits.h rename to CryptoPkg/Library/Include/limits.h diff --git a/CryptoPkg/Include/memory.h b/CryptoPkg/Library/Include/memory.h similarity index 100% rename from CryptoPkg/Include/memory.h rename to CryptoPkg/Library/Include/memory.h diff --git a/CryptoPkg/Include/openssl/opensslconf.h b/CryptoPkg/Library/In= clude/openssl/opensslconf.h similarity index 100% rename from CryptoPkg/Include/openssl/opensslconf.h rename to CryptoPkg/Library/Include/openssl/opensslconf.h diff --git a/CryptoPkg/Include/stdarg.h b/CryptoPkg/Library/Include/stdarg.h similarity index 100% rename from CryptoPkg/Include/stdarg.h rename to CryptoPkg/Library/Include/stdarg.h diff --git a/CryptoPkg/Include/stddef.h b/CryptoPkg/Library/Include/stddef.h similarity index 100% rename from CryptoPkg/Include/stddef.h rename to CryptoPkg/Library/Include/stddef.h diff --git a/CryptoPkg/Include/stdio.h b/CryptoPkg/Library/Include/stdio.h similarity index 100% rename from CryptoPkg/Include/stdio.h rename to CryptoPkg/Library/Include/stdio.h diff --git a/CryptoPkg/Include/stdlib.h b/CryptoPkg/Library/Include/stdlib.h similarity index 100% rename from CryptoPkg/Include/stdlib.h rename to CryptoPkg/Library/Include/stdlib.h diff --git a/CryptoPkg/Include/string.h b/CryptoPkg/Library/Include/string.h similarity index 100% rename from CryptoPkg/Include/string.h rename to CryptoPkg/Library/Include/string.h diff --git a/CryptoPkg/Include/strings.h b/CryptoPkg/Library/Include/string= s.h similarity index 100% rename from CryptoPkg/Include/strings.h rename to CryptoPkg/Library/Include/strings.h diff --git a/CryptoPkg/Include/sys/time.h b/CryptoPkg/Library/Include/sys/t= ime.h similarity index 100% rename from CryptoPkg/Include/sys/time.h rename to CryptoPkg/Library/Include/sys/time.h diff --git a/CryptoPkg/Include/sys/types.h b/CryptoPkg/Library/Include/sys/= types.h similarity index 100% rename from CryptoPkg/Include/sys/types.h rename to CryptoPkg/Library/Include/sys/types.h diff --git a/CryptoPkg/Include/syslog.h b/CryptoPkg/Library/Include/syslog.h similarity index 100% rename from CryptoPkg/Include/syslog.h rename to CryptoPkg/Library/Include/syslog.h diff --git a/CryptoPkg/Include/time.h b/CryptoPkg/Library/Include/time.h similarity index 100% rename from CryptoPkg/Include/time.h rename to CryptoPkg/Library/Include/time.h diff --git a/CryptoPkg/Include/unistd.h b/CryptoPkg/Library/Include/unistd.h similarity index 100% rename from CryptoPkg/Include/unistd.h rename to CryptoPkg/Library/Include/unistd.h diff --git a/CryptoPkg/Library/OpensslLib/process_files.pl b/CryptoPkg/Libr= ary/OpensslLib/process_files.pl index 210811b9ed..716ecbf39a 100644 --- a/CryptoPkg/Library/OpensslLib/process_files.pl +++ b/CryptoPkg/Library/OpensslLib/process_files.pl @@ -214,7 +214,7 @@ print "Done!"; # print "\n--> Duplicating opensslconf.h into Include/openssl ... "; copy($OPENSSL_PATH . "/include/openssl/opensslconf.h", - $OPENSSL_PATH . "/../../../Include/openssl/") || + $OPENSSL_PATH . "/../../Include/openssl/") || die "Cannot copy opensslconf.h!"; print "Done!\n"; =20 --=20 2.12.2.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel