From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 02267438C7; Mon, 15 Jan 2024 03:57:51 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E32544029F; Mon, 15 Jan 2024 03:57:50 +0100 (CET) Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by mails.dpdk.org (Postfix) with ESMTP id E081640272 for ; Mon, 15 Jan 2024 03:57:49 +0100 (CET) Received: from mail.maildlp.com (unknown [172.19.163.174]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4TCxcn2gYyzsVy8; Mon, 15 Jan 2024 10:56:57 +0800 (CST) Received: from dggpeml500024.china.huawei.com (unknown [7.185.36.10]) by mail.maildlp.com (Postfix) with ESMTPS id 0B13B1400E5; Mon, 15 Jan 2024 10:57:48 +0800 (CST) Received: from [10.67.121.161] (10.67.121.161) by dggpeml500024.china.huawei.com (7.185.36.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 15 Jan 2024 10:57:47 +0800 Subject: Re: [PATCH v2 00/16] verify strdup return value From: fengchengwen To: , CC: References: <20231110100117.8350-1-fengchengwen@huawei.com> Message-ID: <2c002a0d-4c6e-8b7a-e3f4-e4ebb04d3ae2@huawei.com> Date: Mon, 15 Jan 2024 10:57:47 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: <20231110100117.8350-1-fengchengwen@huawei.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.121.161] X-ClientProxiedBy: dggems702-chm.china.huawei.com (10.3.19.179) To dggpeml500024.china.huawei.com (7.185.36.10) 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 Friendly ping. On 2023/11/10 18:01, Chengwen Feng wrote: > This patchset mainly fix the return value of strdup not checked which > may lead to segment fault. It also include two commits which fix memory > leak of strdup. > > Chengwen Feng (16): > eal: verify strdup return value > bus/dpaa: verify strdup return value > bus/fslmc: verify strdup return value > bus/vdev: verify strdup return value > dma/idxd: verify strdup return value > event/cnxk: verify strdup return value > net/failsafe: fix memory leak when parse args > net/nfp: verify strdup return value > app/dumpcap: verify strdup return value > app/pdump: verify strdup return value > app/test: verify strdup return value > app/test-crypto-perf: verify strdup return value > app/test-dma-perf: verify strdup return value > app/testpmd: verify strdup return value > examples/qos_sched: fix memory leak when parse args > examples/vhost: verify strdup return value > > --- > v2: fix 0-day warning of app/dumpcap commit. > > app/dumpcap/main.c | 5 ++++ > app/pdump/main.c | 3 +++ > app/test-crypto-perf/cperf_options_parsing.c | 4 ++++ > app/test-dma-perf/main.c | 2 ++ > app/test-pmd/cmdline.c | 2 ++ > app/test/process.h | 5 +++- > drivers/bus/dpaa/dpaa_bus.c | 4 ++++ > drivers/bus/fslmc/fslmc_bus.c | 4 ++++ > drivers/bus/vdev/vdev.c | 4 ++++ > drivers/dma/idxd/idxd_bus.c | 10 ++++++-- > drivers/event/cnxk/cnxk_eventdev.c | 3 +++ > drivers/net/failsafe/failsafe_args.c | 2 +- > drivers/net/nfp/nfp_ipsec.c | 5 ++++ > examples/qos_sched/args.c | 4 +++- > examples/vhost/main.c | 3 +++ > lib/eal/common/eal_common_options.c | 24 ++++++++++++++++++-- > lib/eal/linux/eal_dev.c | 3 +++ > 17 files changed, 80 insertions(+), 7 deletions(-) >