Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/114986/?format=api
{ "id": 114986, "url": "http://patches.dpdk.org/api/patches/114986/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20220815071306.2910599-30-qi.z.zhang@intel.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": "<20220815071306.2910599-30-qi.z.zhang@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20220815071306.2910599-30-qi.z.zhang@intel.com", "date": "2022-08-15T07:12:25", "name": "[29/70] net/ice/base: fix array overflow in add switch recipe code", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "5cd3a4407119dd78ab8dfa0422120dfe4dc3ea48", "submitter": { "id": 504, "url": "http://patches.dpdk.org/api/people/504/?format=api", "name": "Qi Zhang", "email": "qi.z.zhang@intel.com" }, "delegate": { "id": 1540, "url": "http://patches.dpdk.org/api/users/1540/?format=api", "username": "qzhan15", "first_name": "Qi", "last_name": "Zhang", "email": "qi.z.zhang@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20220815071306.2910599-30-qi.z.zhang@intel.com/mbox/", "series": [ { "id": 24307, "url": "http://patches.dpdk.org/api/series/24307/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=24307", "date": "2022-08-15T07:11:56", "name": "ice base code update", "version": 1, "mbox": "http://patches.dpdk.org/series/24307/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/114986/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/114986/checks/", "tags": {}, "related": [], "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 BAA77A00C3;\n\tMon, 15 Aug 2022 01:06:17 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id D525242C26;\n\tMon, 15 Aug 2022 01:03:59 +0200 (CEST)", "from mga11.intel.com (mga11.intel.com [192.55.52.93])\n by mails.dpdk.org (Postfix) with ESMTP id 26B1C42BBD;\n Mon, 15 Aug 2022 01:03:58 +0200 (CEST)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 14 Aug 2022 16:03:57 -0700", "from dpdk-qzhan15-test02.sh.intel.com ([10.67.115.4])\n by fmsmga005.fm.intel.com with ESMTP; 14 Aug 2022 16:03:56 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1660518238; x=1692054238;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=J43I6398FsLaf+0CgJG9oYHwdzs/Fe3qvqKhitBSb1Q=;\n b=MJPLcH6t8HVhJkZQR3WiiLGdKchkaEaDGVM25atItXet3x3LEvHwKunR\n Hu815GBSo7a1XCRmAAUdS+unuJvQQmswVtSASoAzWJQEtQt6HkL7VbZMd\n twSYu96q6bFvyqroKZIb417JDDEU1E1UWk4nJeRBjUOLFsh40tAi4g2on\n Dv/yj9MOfpv+piInr89PDFtAcUnOb9mhakeyBhEWqBHgy7XhLPEfXgrrw\n t6b3ikexQ4azAGgwP130C+q3n+JNEtB1HhG7YrsP04qHEA4NmIF0r/LJg\n wJFT6AEXIIoKvuSdfzkP63x0a4r9Kcs94jNx66xsXfGzbDGezQBaD+PCK g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6400,9594,10439\"; a=\"289427601\"", "E=Sophos;i=\"5.93,237,1654585200\"; d=\"scan'208\";a=\"289427601\"", "E=Sophos;i=\"5.93,237,1654585200\"; d=\"scan'208\";a=\"934296700\"" ], "X-ExtLoop1": "1", "From": "Qi Zhang <qi.z.zhang@intel.com>", "To": "qiming.yang@intel.com", "Cc": "dev@dpdk.org, Qi Zhang <qi.z.zhang@intel.com>, stable@dpdk.org,\n Jesse Brandeburg <jesse.brandeburg@intel.com>", "Subject": "[PATCH 29/70] net/ice/base: fix array overflow in add switch recipe\n code", "Date": "Mon, 15 Aug 2022 03:12:25 -0400", "Message-Id": "<20220815071306.2910599-30-qi.z.zhang@intel.com>", "X-Mailer": "git-send-email 2.31.1", "In-Reply-To": "<20220815071306.2910599-1-qi.z.zhang@intel.com>", "References": "<20220815071306.2910599-1-qi.z.zhang@intel.com>", "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": "The array indexes in this function are used with a zero index in the\nfv_idx table, and with a +1 index in the lkup_idx arrays. The code\nwas using the lookup index for the field vector in only one place in\nthis function, but the code was never used after this point so just\nremove the bad line.\n\nThis was caught by the undefined behavior sanitizer.\n\nFixes: fed0c5ca5f19 (\"net/ice/base: support programming a new switch recipe\")\nCc: stable@dpdk.org\n\nSigned-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>\nSigned-off-by: Qi Zhang <qi.z.zhang@intel.com>\n---\n drivers/net/ice/base/ice_switch.c | 1 -\n 1 file changed, 1 deletion(-)", "diff": "diff --git a/drivers/net/ice/base/ice_switch.c b/drivers/net/ice/base/ice_switch.c\nindex e59d191c46..b8e733f539 100644\n--- a/drivers/net/ice/base/ice_switch.c\n+++ b/drivers/net/ice/base/ice_switch.c\n@@ -7315,7 +7315,6 @@ ice_add_sw_recipe(struct ice_hw *hw, struct ice_sw_recipe *rm,\n \t\tlast_chain_entry->chain_idx = ICE_INVAL_CHAIN_IND;\n \t\tLIST_FOR_EACH_ENTRY(entry, &rm->rg_list, ice_recp_grp_entry,\n \t\t\t\t l_entry) {\n-\t\t\tlast_chain_entry->fv_idx[i] = entry->chain_idx;\n \t\t\tbuf[recps].content.lkup_indx[i] = entry->chain_idx;\n \t\t\tbuf[recps].content.mask[i++] = CPU_TO_LE16(0xFFFF);\n \t\t\tice_set_bit(entry->rid, rm->r_bitmap);\n", "prefixes": [ "29/70" ] }