On 11/1/2018 7:46 PM, Bruce Richardson wrote: > On Wed, Oct 31, 2018 at 06:35:11AM +0000, Jerin Jacob wrote: >> -----Original Message----- >>> Date: Wed, 31 Oct 2018 00:39:54 +0000 >>> From: Fiona Trahe <fiona.tr...@intel.com> >>> To: dev@dpdk.org >>> CC: tho...@monjalon.net, akhil.go...@nxp.com, tomaszx.jozw...@intel.com, >>> jerin.ja...@caviumnetworks.com, Fiona Trahe <fiona.tr...@intel.com> >>> Subject: [PATCH] compress/qat: fix out-of-bounds error >>> X-Mailer: git-send-email 1.7.0.7 >>> >>> >>> QAT array for sgls in intermediate buffer structure >>> was #defined to 1, but setup code hardcoded as if 2 buffers >>> so causing out of bounds write. Reworked to loop correctly >>> using #define. >>> >>> Fixes: a124830a6f00 ("compress/qat: enable dynamic huffman encoding") >>> >>> Reported-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> >>> Signed-off-by: Fiona Trahe <fiona.tr...@intel.com> >> clang build is not reproducible with this patch. > s/not/now/ :-) >> Tested-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> >> > I can also confirm that clang builds which were failing without this patch > are now building ok on Fedora 29. > > Acked-by: Bruce Richardson <bruce.richard...@intel.com> > Applied to dpdk-next-crypto
Thanks.