From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web12.1668.1601593296434501426 for ; Thu, 01 Oct 2020 16:01:36 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=KTBln7OJ; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: nathaniel.l.desimone@intel.com) IronPort-SDR: UTZ5jsejYP9cOou6G0A3gfKpXzMyqbNMWBwYEaCePOKX64Mb0EyrErMsDLm8QEKGD+f5KKgMzs xLcUk4DjM2OQ== X-IronPort-AV: E=McAfee;i="6000,8403,9761"; a="150654525" X-IronPort-AV: E=Sophos;i="5.77,325,1596524400"; d="scan'208";a="150654525" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Oct 2020 16:01:33 -0700 IronPort-SDR: tGy+lNZP1x2M28d/cw2mPURpa+BiB4psVT8j6rl+ByXhB0TAt8ZG3eWr97Yx3kWxkqJgvo0LWU Mtrwvai8jaXA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,325,1596524400"; d="scan'208";a="416242122" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by fmsmga001.fm.intel.com with ESMTP; 01 Oct 2020 16:01:33 -0700 Received: from fmsmsx603.amr.corp.intel.com (10.18.126.83) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Thu, 1 Oct 2020 16:01:32 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5 via Frontend Transport; Thu, 1 Oct 2020 16:01:32 -0700 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.170) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.1713.5; Thu, 1 Oct 2020 16:01:32 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ctsSCwvyrnEcwws9MAb5R6a27jZSg4rtvskdoUkYNYBBmNUgNzCm/PA9Wv7Pjkj+JVCZ9bpaEQkqHd9yuDmPHjicqTXttyERH/vbYrqJRbc0WJKY5EdDSaHPMpXTCnNIPN9iUd1wX/SBLSa+vzKi7FLSY6Mco2sO4CEfYmOvd9s5bVUP09pirdqQWLpfZrtvQjU480clPNDe9r78pySXxuqfFCuuG89NoigYo+UEwRDpOgf+xsEJePSIKeflHYLFH0Uvi5S913BfIVHq6NU5VbLiT0z2DnOJjQH9ZoECMGc/mqcMZxXfIzbDybh5bzZjI/w0teGtGqMjz+dUg0eu8g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=94KOb7k4eJeU6cDmXKr0PNtuwMDxS0bngjDDXNkD+NE=; b=Fae1Ta0qoNvHc4sKm4R+YRsR6+1likuIiJJy5Le2qO+6tbhsQprRc8vIYbfActdacerBnM2Aa2KjUZ2eSPWkIUaJiHx/R1DeIFL5GrYU55uPt/UX2ICfeJnwMfMZcRfcqOL15qnRVEj9lXM8Ccs7ZunoxRGugiyJzAulu0Oen8HNYky9MOXUKZ5CcJulGZBgeEbKrsq3Gwf6qBDNzEF/Nq7sWveCffmj0MqTqYIm0lRnD/xGivhF+gg23LVxCd6mywZiA+H6XVVbIID7zEUowW8DmnphQlqVXwkUTZv5vcqoTL23JGinLR/roPAw6VuJ94/uTuUMTbAgHYDF7kWWhg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=94KOb7k4eJeU6cDmXKr0PNtuwMDxS0bngjDDXNkD+NE=; b=KTBln7OJJDMxpD63RGZSYWaHQuv+OXo46ju/124922QDn0YCTq+Z4LS+vnq9lbkAnjaArwXPcDnscmEc6zg5RHCtOu1hEqFRsR+ZBGINSVbPilPNVX5VX6lBSb+v+h0w3eZCifPOVSRo3Ud2RWcmDEHVoIwk1JLZI8S3lJND/9Q= Received: from MWHPR1101MB2160.namprd11.prod.outlook.com (2603:10b6:301:5b::12) by MWHPR1101MB2207.namprd11.prod.outlook.com (2603:10b6:301:58::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3412.26; Thu, 1 Oct 2020 23:01:31 +0000 Received: from MWHPR1101MB2160.namprd11.prod.outlook.com ([fe80::75c9:81e9:2646:791c]) by MWHPR1101MB2160.namprd11.prod.outlook.com ([fe80::75c9:81e9:2646:791c%4]) with mapi id 15.20.3433.036; Thu, 1 Oct 2020 23:01:31 +0000 From: "Nate DeSimone" To: "devel@edk2.groups.io" , "Desimone, Nathaniel L" CC: "Chiu, Chasel" , Liming Gao , "Dong, Eric" Subject: Re: [edk2-devel] [edk2-platforms] [PATCH V1 0/2] MinPlatformPkg: Minor fixes in CompressLib Thread-Topic: [edk2-devel] [edk2-platforms] [PATCH V1 0/2] MinPlatformPkg: Minor fixes in CompressLib Thread-Index: AQHWlr7vQilgsY94w0uLvWm9OrAVYKmDX95A Date: Thu, 1 Oct 2020 23:01:30 +0000 Message-ID: References: <16396781EA545897.9810@groups.io> In-Reply-To: <16396781EA545897.9810@groups.io> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMGM4YzM4YzktM2RmNi00YjRkLThjMjMtNGZkZTgzZjgwMDgwIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoicHNUblB0dFZSZkg4bzhwZExRNFBISVdLd1NcL1hITkdpcG1ieGo0OXlaRTVXV1wvUVZOYnEzb1lhXC9FTnB2V3BXRCJ9 dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows x-ctpclassification: CTP_NT authentication-results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=intel.com; x-originating-ip: [50.53.190.176] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 47f6e0e0-e1f4-4a94-fff4-08d8665defa7 x-ms-traffictypediagnostic: MWHPR1101MB2207: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:506; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: rTa5aGYK+ByuRrZ+Kk9xa4DgCM2xsxOL5dAaK14UTywJbGn7e6NTTwIKV0ppzOqtqKctuzfBM8rI2VZkC7sQKprjti7isbVY/o7ClYgYgNr8jXadShDg6q45RoTgW1xn2NBPlMojfWrJQcXZGzy3h219rBwzurJXhkrEzeYQ9IgfFVvS8ACE0DIRdQeHIa4onR+CP4Ol2w5V3VfnzyKaTj2z5UGo72QxSgGW16RrFnJh//eJdnE8+w3hKagMnkklqdjYI2AfoO5esqHoGDB9ulwfuc0VcgxK0whicVr5V6jmNXSbRSO3TefvYvrqQt18ddcEKRWkhTrtNpLPl23cOpjY7eK/E2KzN6aMmTBhytvsogi37pgALY9CFEU14ndaVqE8OyOrlh48QhSMheo/FSyf0GMhcL404t8xVvsnRMVdzcKQ3YvoRJrWlH0cdQme x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR1101MB2160.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(39860400002)(136003)(396003)(346002)(376002)(52536014)(8936002)(66946007)(64756008)(66446008)(66476007)(76116006)(71200400001)(107886003)(5660300002)(966005)(66556008)(55016002)(4326008)(83380400001)(110136005)(33656002)(316002)(86362001)(8676002)(53546011)(6506007)(7696005)(54906003)(186003)(2906002)(478600001)(26005)(9686003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: uGDOdaoHVNqLG1bHuIV/wKgLE2KB+pGO9+/aKjhB8GuiVOW0chOXP7KhxfSyHhNYBoGkp0DhyUYaQ5ghI1KgD2MnYv6SITdRzbdPIKL0OoTMciNMeYu6Tg+dCnN10g87tom5CWEENJXfiYJ50RLT/KY8Qp/wCsTNOlt+gryKE/NtU7YmvJnu5U2g+DXMqph5nN0V6HQXVk3zC4GTv28wBVtTzo21c8l7sZxmnG2MGAlKA90TuEXpkvVhvb8p7eCktKmG+mMcN6gUy8ZZGmZKL9R5f4vOUoH0VIyBQDvOIPKIOIthOHJgBWAUf9ZxtDtNqqNEUiXIp53aWdJiP2ijwtraJHkgyITcbOj/IGk3Sj9mwSm8q4EkYNqKmsLuyuMula5klG/hAoP9FR4QiGvT3PQvjCdNdaoAGYj9DG6qj2LdOFUUTzpsxYzqehoIhEAnLPUJx627w3JApQrCCzm4U0Ann5urIjlTKz/zjuNH2RbegjJA/YxkuDa3VmLPAfrE2InpdaGwXODk950fck9PhXhnn/+zNBhuMLcbWiJCKCTu0lBTcJePGFUOCujCFnqu+8OWUX0IiluEPcNBmnj2Dr/TUK3uY9EjIgy1Bf1O/I0meoM5b6mIB8hVTvIs/AiljFzZ6DYqMK13BUpkakne5w== MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2160.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 47f6e0e0-e1f4-4a94-fff4-08d8665defa7 X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Oct 2020 23:01:30.9492 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: jhB/+wXRRMsGWfHSzat9DNhXmWmxaLfAyxVokHb7ahvmSlUFh/JdI6mEMGEfwhYyO1dJF41uEj7nNRlHWjS8Q9cWO6kA5ydPl84Neh8Ng28= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1101MB2207 Return-Path: nathaniel.l.desimone@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Series pushed: ~b9f61c0..46df7b2 > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Nate > DeSimone > Sent: Tuesday, September 29, 2020 5:15 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Liming Gao > ; Dong, Eric > Subject: [edk2-devel] [edk2-platforms] [PATCH V1 0/2] MinPlatformPkg: > Minor fixes in CompressLib >=20 > Current code only has bounds checks in ASSERT macros. > They are also needed in release mode where ASSERT is not used. > Also adds coding style cleanups in CompressLib >=20 > Signed-off-by: Nate DeSimone > Cc: Chasel Chiu > Cc: Nate DeSimone > Cc: Liming Gao > Cc: Eric Dong >=20 > Nate DeSimone (2): > MinPlatformPkg: Add missing bounds checks to CompressLib > MinPlatformPkg: Coding style cleanups in CompressLib >=20 > .../Library/CompressLib/CompressLib.c | 187 ++++++++++-------- > .../Library/CompressLib/CompressLib.inf | 4 +- > 2 files changed, 103 insertions(+), 88 deletions(-) >=20 > -- > 2.27.0.windows.1 >=20 >=20 >=20 >=20