From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id B2B6A1B2B2 for ; Wed, 17 Jan 2018 18:23:42 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Jan 2018 09:23:41 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,372,1511856000"; d="scan'208";a="22455990" Received: from irsmsx103.ger.corp.intel.com ([163.33.3.157]) by fmsmga004.fm.intel.com with ESMTP; 17 Jan 2018 09:23:40 -0800 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.167]) by IRSMSX103.ger.corp.intel.com ([169.254.3.138]) with mapi id 14.03.0319.002; Wed, 17 Jan 2018 17:23:40 +0000 From: "De Lara Guarch, Pablo" To: "Trahe, Fiona" , "Jozwiak, TomaszX" CC: "dev@dpdk.org" , "Jozwiak, TomaszX" Thread-Topic: [dpdk-dev] [PATCH] crypto/qat: fix out-of-bounds compiler error Thread-Index: AQHTjghAnpJH9wklUEun5BfURLOm7KN0/tSAgANN+nA= Date: Wed, 17 Jan 2018 17:23:39 +0000 Message-ID: References: <1516024409-7659-1-git-send-email-tomaszx.jozwiak@intel.com> <348A99DA5F5B7549AA880327E580B435892F87FE@IRSMSX101.ger.corp.intel.com> In-Reply-To: <348A99DA5F5B7549AA880327E580B435892F87FE@IRSMSX101.ger.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjE2OTk5OWEtNGQwMC00ZWRkLThkMWQtODQ3MzMyMzljYzk4IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6ImVxOE9xT0VLa2xIa0xpRGhJdUhLUUN1Tld0RDQrdGpqV3Q0UzlPSUlxNWM9In0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] crypto/qat: fix out-of-bounds compiler error X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Jan 2018 17:23:43 -0000 > -----Original Message----- > From: Trahe, Fiona > Sent: Monday, January 15, 2018 2:31 PM > To: Jozwiak, TomaszX ; De Lara Guarch, Pablo > > Cc: dev@dpdk.org; Jozwiak, TomaszX ; Trahe, > Fiona > Subject: RE: [dpdk-dev] [PATCH] crypto/qat: fix out-of-bounds compiler > error >=20 >=20 >=20 > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Tomasz Jozwiak > > Sent: Monday, January 15, 2018 1:53 PM > > To: De Lara Guarch, Pablo > > Cc: dev@dpdk.org; Jozwiak, TomaszX > > Subject: [dpdk-dev] [PATCH] crypto/qat: fix out-of-bounds compiler > > error > > > > This commit fixes > > - bpi_cipher_encrypt to prevent before 'array subscript is > > above array bounds' error > > - bpi_cipher_decrypt to prevent before 'array subscript is > > above array bounds' error > > - right cast from qat_cipher_get_block_size function. > > This function can return -EFAULT in case of any error, > > and that value must be cast to int instead of uint8_t > > - typo in error message > > > > A performance improvement was added to bpi_cipher_encrypt and > > bpi_cipher_decrypt as well. > > > > Fixes: d18ab45f7654 ("crypto/qat: support DOCSIS BPI mode") > > > > Signed-off-by: Tomasz Jozwiak >=20 > Acked-by: Fiona Trahe Hi Tomasz, >>From what I see in this patch, it is making various changes, which are not related (such as the typo in the error message). Could you split the patch into several ones? You can leave the ack from Fio= na, and make sure that the patch that fixes the out-of-bounds problem also CC s= table@dpdk.org. Thanks, Pablo