From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liang Ma Subject: [PATCH 2/2] event/opdl: fix dereference before null check Date: Wed, 24 Jan 2018 15:05:51 +0000 Message-ID: <1516806351-151804-2-git-send-email-liang.j.ma@intel.com> References: <1516806351-151804-1-git-send-email-liang.j.ma@intel.com> Cc: harry.van.haaren@intel.com, bruce.richardson@intel.com, peter.mccarthy@intel.com, jerin.jacob@caviumnetworks.com To: dev@dpdk.org Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id EAD811B198 for ; Wed, 24 Jan 2018 16:05:41 +0100 (CET) In-Reply-To: <1516806351-151804-1-git-send-email-liang.j.ma@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Fixes: 4236ce9bf5bf ("event/opdl: add OPDL ring infrastructure library") Coverity issue: 257022 Signed-off-by: Liang Ma --- drivers/event/opdl/opdl_ring.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/event/opdl/opdl_ring.c b/drivers/event/opdl/opdl_ring.c index 7e16d4d..39dc41d 100644 --- a/drivers/event/opdl/opdl_ring.c +++ b/drivers/event/opdl/opdl_ring.c @@ -550,6 +550,10 @@ opdl_stage_claim_multithread(struct opdl_stage *s, void *entries, uint32_t i = 0, offset; uint8_t *entries_offset = (uint8_t *)entries; + if (seq == NULL) { + PMD_DRV_LOG(ERR, "Invalid seq PTR"); + return 0; + } offset = opdl_first_entry_id(*seq, s->nb_instance, s->instance_id); num_entries = offset + (s->nb_instance * num_entries); @@ -561,8 +565,8 @@ opdl_stage_claim_multithread(struct opdl_stage *s, void *entries, entries_offset += t->slot_size; i++; } - if (seq != NULL) - *seq = old_head; + + *seq = old_head; return i; } -- 2.7.5