Cover Detail
Show a cover letter.
GET /api/covers/108030/?format=api
{ "id": 108030, "url": "http://patches.dpdk.org/api/covers/108030/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/tencent_3C112E11A018311EA7D3D908771D89EFBF07@qq.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<tencent_3C112E11A018311EA7D3D908771D89EFBF07@qq.com>", "list_archive_url": "https://inbox.dpdk.org/dev/tencent_3C112E11A018311EA7D3D908771D89EFBF07@qq.com", "date": "2022-02-22T18:17:58", "name": "[00/20] fix memory leaks in error handling", "submitter": { "id": 2458, "url": "http://patches.dpdk.org/api/people/2458/?format=api", "name": "Weiguo Li", "email": "liwg06@foxmail.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/tencent_3C112E11A018311EA7D3D908771D89EFBF07@qq.com/mbox/", "series": [ { "id": 21799, "url": "http://patches.dpdk.org/api/series/21799/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=21799", "date": "2022-02-22T18:17:58", "name": "fix memory leaks in error handling", "version": 1, "mbox": "http://patches.dpdk.org/series/21799/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/108030/comments/", "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "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])\n\tby inbox.dpdk.org (Postfix) with ESMTP id DEC9AA034E;\n\tTue, 22 Feb 2022 19:18:34 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 833D740DF6;\n\tTue, 22 Feb 2022 19:18:34 +0100 (CET)", "from out203-205-221-235.mail.qq.com (out203-205-221-235.mail.qq.com\n [203.205.221.235])\n by mails.dpdk.org (Postfix) with ESMTP id D608140DF4\n for <dev@dpdk.org>; Tue, 22 Feb 2022 19:18:32 +0100 (CET)", "from localhost.localdomain ([111.193.130.237])\n by newxmesmtplogicsvrszc6.qq.com (NewEsmtp) with SMTP\n id 49223207; Wed, 23 Feb 2022 02:18:18 +0800" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com;\n s=s201512; t=1645553909;\n bh=BbEvK2SNRoVnBKbnGvXZnjoc5O7HHTi+WbMMPrq5qic=;\n h=From:To:Cc:Subject:Date;\n b=LRBjnOaT1Xq/W51HkgOd4nQoZbWRp1TaGbi4emTCOPgkq3bJlqVbauYsLTNrrIC+h\n k2ZwSkwY8h8wefvs3pN7XMDrrREbgADsjDQMlrD2mMHbPgX1nqly/b0sJn+A/ILrgb\n 8w5MD+YSJUq0JEqub3ZuxcUdDw6eT1id1434c6UA=", "X-QQ-mid": "xmsmtpt1645553898tsmh9awn9", "Message-ID": "<tencent_3C112E11A018311EA7D3D908771D89EFBF07@qq.com>", "X-QQ-XMAILINFO": "OZ7HbAk7YCRi34tNTEdTEfFRRt+ZC1b4cSKSSjpZQ9NrjCdyw83yiSXcI8Kj9P\n 8R/EaXdcdzFxO+7zcTcUNAkbmRApeOCMwSM0ZrxbV+sJ5nojOPzrGJNuV897bRcZKHr6xuUS9OWp\n 85BKkn8oS5gFBFZUa5BnKkTv55OipXsHNgPcWOfdvWcGuSDSqxgURrkacZqH8VBcYA3BtFSgIgHA\n DkmVPxACfssS4M4vFb46owBk3WRQClOL4d+Np1bFVO9C6pkavgi0+fcXWDVCfHOxjbXtkG7HxJbS\n 7MZZDEuTTmLAx+PVLWO0GXzB5S0K5MxADr1qoxrH5GHmY1EMn4vKw7LES2wBKBWCjY8qfprAbcag\n 8FVSSXJ+rXD6nst86S+jYiDTKuRGRx7rnHkkQovXUn/+Lks3X305qzcNKQ7myZXOE7IbBb4S77IY\n sghRM07bn6DtljwvapIurKRfNcT70MTDAHfLzeHGgtlYuedPQI7iLgIQj4hJWzpaFTJDuDHUlMKA\n ClZKRwSY1Ma5z5nGh+/l7G/4vosjtWYI60IONumwSwtfcD3VaMOeZpknnxJzaRkJFxOW43z6B+Jk\n YFFTkthgU552E4IqjndPYz8vgJyW0djxSSQvucaM5rbYEQ6/SrfyfySp1Cp6dICTrPt0GbhRfm5h\n fIkuD52mGat02GOSjAc4UyZ4FzpqOlO4Pg8eep+7S2NvyroNzXF59NOUnDTLCeJta/a8rCGnH0ZP\n QPLciKtATq6FNIsfGyNkJ19Wvhgdlh1Yk3bkUP9BGFUCp9MN2uYTVRuXUGLTedCJXdiDQBgYssth\n eXFHvRHJRESPgr6WwGhgrkIRoX5CrvCOOImryxUOvAzpkmrkYhU1UHOgvQOdjDoa0=", "From": "Weiguo Li <liwg06@foxmail.com>", "To": "dev@dpdk.org", "Cc": "adypodoman@gmail.com, xiaoyun.li@intel.com, tianfei.zhang@intel.com,\n bruce.richardson@intel.com, ivan.malov@oktetlabs.ru, jgrajcia@cisco.com,\n hkalra@marvell.com, ying.a.wang@intel.com, ting.xu@intel.com,\n simei.su@intel.com, qiming.yang@intel.com, motih@mellanox.com,\n shreyansh.jain@nxp.com, skoteshwar@marvell.com, stephen@networkplumber.org,\n kalesh-anakkur.purayil@broadcom.com, somnath.kotur@broadcom.com,\n declan.doherty@intel.com, gakhil@marvell.com, nicolas.chautru@intel.com", "Subject": "[PATCH 00/20] fix memory leaks in error handling", "Date": "Wed, 23 Feb 2022 02:17:58 +0800", "X-OQ-MSGID": "<cover.1645551559.git.liwg06@foxmail.com>", "X-Mailer": "git-send-email 2.25.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "This series fix some memory leaks in error handling.\n\nI write a coccinelle script to detect these issues, and\nfix them after exclude a small number of false positives.\n\nFYI, the script is as follows:\n//\n// Find possible memory leaks in error handling\n// Reference: coccinellery/alloc9/kmalloc9.cocci\n//\n@memory_leak_in_error_handling @\nlocal idexpression x;\nexpression E;\nstatement S;\nidentifier dpdk_malloc = { rte_malloc, rte_zmalloc, rte_realloc, rte_malloc_socket,\n rte_zmalloc_socket, rte_realloc_socket, rte_pktmbuf_alloc, opae_malloc,\n strdup, strndup, malloc, realloc, calloc };\nidentifier dpdk_free = { rte_free, free, bnx2x_rx_queue_release, bnx2x_tx_queue_release,\n rte_pktmbuf_free, opae_free, ntb_rxq_release, ntb_txq_release, ice_rx_queue_release,\n ice_tx_queue_release };\ntype T;\n@@\n(\n /* skip this pattern to reduce false positives */\n x = dpdk_malloc(...); ... if (...) { ... \\( return x; \\| return 0; \\) }\n|\n* x = dpdk_malloc(...);\n ...\n if (x == NULL) S\n ... when != E = x\n when != dpdk_free (..., \\( x \\| (T)x \\| &x \\), ...)\n when forall\n if (...) {\n <+... when != E = x\n when != dpdk_free (..., \\( x \\| (T)x \\| &x \\), ...)\n when forall\n* return ...;\n ...+>\n }\n)\n\nWeiguo Li (20):\n baseband/acc100: fix a memory leak in acc100 queue setup\n common/dpaax: fix a memory leak in iterate dir\n crypto/dpaa2_sec: fix memory leaks in error handlings\n crypto/qat: fix a memory leak when set encrypt key fail\n net/bnxt: fix a memory leak in error handling\n net/bnxt: fix 'ctx' memory leak when new malloc fail\n net/bnx2x: add clean up for 'rxq' to avoid a memory leak\n net/cnxk: free 'node' memory when node add fail\n net/dpaa: fix a memory leak when validation fail\n net/failsafe: fix a memory leak in error handling\n net/iavf: fix a memory leak in error handling\n net/ice: goto clean up lable to avoid memory leak\n net/ice: fix memory leaks in error handlings\n net/ice: avoid fix memory leaks in register parser\n net/memif: fix some memory leaks in error handlings\n net/sfc: fix a memory leak in error handling\n net/vmxnet3: fix memory leaks in error handlings\n raw/ifpga/base: fix memory leaks in error handlings\n raw/ntb: fix some memory leaks in error handlings\n regex/mlx5: fix a memory leak in error handling\n\n drivers/baseband/acc100/rte_acc100_pmd.c | 8 +++---\n drivers/common/dpaax/dpaa_of.c | 4 ++-\n drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c | 3 ++\n drivers/crypto/qat/qat_sym_session.c | 9 +++---\n drivers/net/bnx2x/bnx2x_rxtx.c | 1 +\n drivers/net/bnxt/bnxt_hwrm.c | 1 +\n drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c | 1 +\n drivers/net/cnxk/cnxk_tm.c | 1 +\n drivers/net/dpaa/dpaa_rxtx.c | 1 +\n drivers/net/failsafe/failsafe_ops.c | 1 +\n drivers/net/iavf/iavf_generic_flow.c | 1 +\n drivers/net/ice/ice_acl_filter.c | 2 +-\n drivers/net/ice/ice_generic_flow.c | 9 +++---\n drivers/net/ice/ice_hash.c | 30 ++++++++++++-------\n drivers/net/memif/rte_eth_memif.c | 32 ++++++++++++++-------\n drivers/net/sfc/sfc.c | 4 ++-\n drivers/net/vmxnet3/vmxnet3_rxtx.c | 8 ++++++\n drivers/raw/ifpga/base/ifpga_enumerate.c | 10 +++++--\n drivers/raw/ifpga/base/opae_eth_group.c | 1 +\n drivers/raw/ifpga/base/opae_i2c.c | 5 +++-\n drivers/raw/ntb/ntb.c | 9 +++---\n drivers/regex/mlx5/mlx5_rxp.c | 4 ++-\n 22 files changed, 100 insertions(+), 45 deletions(-)" }