From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] pcap: Fixed bug in eth_pcap_rx function Date: Wed, 13 Aug 2014 01:29:39 +0200 Message-ID: <2092493.KHLGrS08OS@xps13> References: <1407233900-9734-1-git-send-email-pablo.de.lara.guarch@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "De Lara Guarch, Pablo" Return-path: In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" > > Normally, bufs[i] stores the mbuf pointer, the index of buf[i] > > is the loop count i, but if header.len > buf_size, DPDK will > > free the mbuf, but the loop count i still increases, so some > > of the items in bufs[] might be NULL ponter, causing a potential > > DPDK core. Using num_rx as the index for bufs[] solves the problem. > > > > Signed-off-by: Pablo de Lara > > Tested-by: Jiajia SunX Acked-by: Thomas Monjalon Applied for version 1.7.1 Thanks -- Thomas