From mboxrd@z Thu Jan 1 00:00:00 1970 From: Taehee Yoo Subject: [PATCH bpf] xdp: add NULL pointer check in __xdp_return() Date: Sat, 21 Jul 2018 01:04:45 +0900 Message-ID: <20180720160445.17475-1-ap420073@gmail.com> Cc: bjorn.topel@intel.com, brouer@redhat.com, netdev@vger.kernel.org, Taehee Yoo To: daniel@iogearbox.net, ast@kernel.org Return-path: Received: from mail-pg1-f194.google.com ([209.85.215.194]:44375 "EHLO mail-pg1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732141AbeGTQxz (ORCPT ); Fri, 20 Jul 2018 12:53:55 -0400 Received: by mail-pg1-f194.google.com with SMTP id r1-v6so7255564pgp.11 for ; Fri, 20 Jul 2018 09:04:59 -0700 (PDT) Sender: netdev-owner@vger.kernel.org List-ID: rhashtable_lookup() can return NULL. so that NULL pointer check routine should be added. Fixes: 02b55e5657c3 ("xdp: add MEM_TYPE_ZERO_COPY") Signed-off-by: Taehee Yoo --- net/core/xdp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/core/xdp.c b/net/core/xdp.c index 9d1f220..1c12bc7 100644 --- a/net/core/xdp.c +++ b/net/core/xdp.c @@ -345,7 +345,8 @@ static void __xdp_return(void *data, struct xdp_mem_info *mem, bool napi_direct, rcu_read_lock(); /* mem->id is valid, checked in xdp_rxq_info_reg_mem_model() */ xa = rhashtable_lookup(mem_id_ht, &mem->id, mem_id_rht_params); - xa->zc_alloc->free(xa->zc_alloc, handle); + if (xa) + xa->zc_alloc->free(xa->zc_alloc, handle); rcu_read_unlock(); default: /* Not possible, checked in xdp_rxq_info_reg_mem_model() */ -- 2.9.3