From patchwork Mon Jan 24 17:46:39 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Hemminger X-Patchwork-Id: 106401 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4ADACA04AD; Mon, 24 Jan 2022 18:51:23 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4E3F8428EC; Mon, 24 Jan 2022 18:48:13 +0100 (CET) Received: from mail-pf1-f171.google.com (mail-pf1-f171.google.com [209.85.210.171]) by mails.dpdk.org (Postfix) with ESMTP id E7929428D4 for ; Mon, 24 Jan 2022 18:48:09 +0100 (CET) Received: by mail-pf1-f171.google.com with SMTP id p37so16669988pfh.4 for ; Mon, 24 Jan 2022 09:48:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=zkr0ICvLVL2t/HP9KwQdbKbxig7p/kOYPIjwjT/68u8=; b=S7ZYATxw7Trng5NDXta7z7kdds2n5J/MXj+OD2/MkoLooGB7ZbBtVmwsTpY2u0gnSl j/4AollHsmZghEbtDzFDfjUSODgcFvulsIVlkeDHw+9q/cFm7ClipaWHfP0nGqWnYV02 24Te++IFmlalPO/MoVwKpbax7f2ll7Ii4rBlcQv/JkODcccIxnieTzNxp3NDxnlbVZMG JSUMcKPW2owWYZne/YNaPmg6HuaZ/G9JjkptHDni0H2hlBBHqH0huE1Ihir0op8Xjzqv W79rYWN5E3b3bHtO+YYr4KEH/tHdGQaAJrxJjZRcMsCFaF/D1eFOToW/nHkRJZpM7GHL FlqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=zkr0ICvLVL2t/HP9KwQdbKbxig7p/kOYPIjwjT/68u8=; b=tICyTZeECQJ4+7v2daCguSF4G8sa93kjTXzqEtJ4OwpvlReFnR/+rMoXNYMkpGdw9N yIsvwszoPAJ/ujHQdiIriJa4YYQudjzfrdN+Etkk15u6oiVVqT8U+Fjzxg6QdXRe/c5t eKLnTNG8PqKryGMVND4pN1kPqOG7qxvAs5dXyFtAcGyrZSNG9iPgflPBdYDxS8aLU3ks kQg4YkdwPhlaQw42VTRI/LywAxMPNhGaAegHklF2Q0o+ShyGWMUmT8UPqzZNxbja1UU5 8yJ1i/kz53Xoa6A0zoo5RTRvBRFspyPvmPAaz5fRLSrYdC+UNhfiYbtqCWOFl0Mk2mHu P70w== X-Gm-Message-State: AOAM532syIcCk0viKrGwxvBkMK01+ZZIVTV7m58p2o+zJPXydCXI/rqD Rj3S3+8o+VCR+c9StspHlIhjf75zPAxUQw== X-Google-Smtp-Source: ABdhPJyLPeKKIqZTFBHu+r++BDobSlSq0zbRn997iQo88n1de8PMrPUN4SKvlgiBNYpRip6ZRuLWjA== X-Received: by 2002:a63:2b01:: with SMTP id r1mr5312505pgr.182.1643046488934; Mon, 24 Jan 2022 09:48:08 -0800 (PST) Received: from hermes.local (204-195-112-199.wavecable.com. [204.195.112.199]) by smtp.gmail.com with ESMTPSA id q7sm15835082pfs.37.2022.01.24.09.48.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Jan 2022 09:48:08 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger , Chas Williams , "Min Hu (Connor)" Subject: [PATCH v2 43/83] net/bonding: remove unnecessary NULL checks Date: Mon, 24 Jan 2022 09:46:39 -0800 Message-Id: <20220124174719.14417-44-stephen@networkplumber.org> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20220124174719.14417-1-stephen@networkplumber.org> References: <20220124000518.319850-1-stephen@networkplumber.org> <20220124174719.14417-1-stephen@networkplumber.org> MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Remove redundant NULL pointer checks before free functions found by nullfree.cocci Signed-off-by: Stephen Hemminger Acked-by: Min Hu (Connor) --- drivers/net/bonding/rte_eth_bond_pmd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c index 84f4900ee52c..963280828a65 100644 --- a/drivers/net/bonding/rte_eth_bond_pmd.c +++ b/drivers/net/bonding/rte_eth_bond_pmd.c @@ -2148,8 +2148,7 @@ bond_ethdev_close(struct rte_eth_dev *dev) */ rte_mempool_free(internals->mode6.mempool); - if (internals->kvlist != NULL) - rte_kvargs_free(internals->kvlist); + rte_kvargs_free(internals->kvlist); return 0; }