From patchwork Tue Oct 10 06:17:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 132454 X-Patchwork-Delegate: ferruh.yigit@amd.com 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 3319742348; Tue, 10 Oct 2023 08:18:03 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 42E5740647; Tue, 10 Oct 2023 08:17:53 +0200 (CEST) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2131.outbound.protection.outlook.com [40.107.237.131]) by mails.dpdk.org (Postfix) with ESMTP id 62551402ED; Tue, 10 Oct 2023 08:17:51 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m5Q1hPVv3vg8+DY04Aimuxr28p3l8678m/RtePxtX6IDufaf+MXb1DD9iZnfYwjeZbSK89066JUTDmri1FNl1+Adbgnx/nPh24Jr4/ZOT511BIBAeXqQrWKZEkBSBWvv65IbCruWVsTbRe77r4eNi/nwVKX7yc8dI8ZBHqqZxBj2S0H3T8fo4rcv3meZCW3NsG6f+u9s9YLVK/oLxU36BuRUIcxR7XmMTlhvwdgOViOyY6pHHafeH9sYtDzcot71z8h/xSBZ9BdYwALAG4T8fufVus3R3akC+Qrb2ItHXIYsrjrHovSZCJAINLxK1bw3sxdrUkP1WecpW7KIKikLnw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0zispk8LqIxqTgwPA4y0RMIKingI/al6FObor/+/E8c=; b=l+fpVbmoHHzG+KUDiw8Lzl2v7AcZGXdGwF0ARpbNFEairHo87VnL2SQgR7jEIAs3/5MrSGObcX6Nc/NM3VmI1wId9rTLYEtXcZEkTVNxWfDgItsgFqweFzWX0mQ6oZ+yzYZbsDcwezrwNtfrlLTgNLUO9eV2uejp+J2GK7O12+dTPMb3Qrh91YBqa28OfhwOXJqHIv88KifN9izL86sCQUvTK7gqqxTJ/aQERCrIp58jzveogv4A/wrh8VynngfeM01a6UAKil53fwOubl6sI/4e2fGqgjZsv9fZEcxVfRlRHfjUZoRBNJbnuL89gfJRJM4Gbjt9oAbttWUQpTIO9A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0zispk8LqIxqTgwPA4y0RMIKingI/al6FObor/+/E8c=; b=daDD9L7llcHo8LoRRq4YRG0JsFMc+D28bJRBA2U1u0u383VxMFR3SFV68e9k9R1GGTtTQ6CZTnfc3sLTnjSpRGYEdp6fnSbJvetBHTI8+X0/HUQTbHa1FLegLOMrLGj1dIBrSlxMs0voixjMD0EJwg5jQP7iIw1BNzlkplNmcHg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM8PR13MB5192.namprd13.prod.outlook.com (2603:10b6:8:6::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6863.38; Tue, 10 Oct 2023 06:17:49 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::2d5:f7d6:ecde:c93a]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::2d5:f7d6:ecde:c93a%4]) with mapi id 15.20.6863.032; Tue, 10 Oct 2023 06:17:49 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, Long Wu , chaoyong.he@corigine.com, stable@dpdk.org, Peng Zhang Subject: [PATCH 2/2] net/nfp: fix illegal memory accesses Date: Tue, 10 Oct 2023 14:17:27 +0800 Message-Id: <20231010061727.205886-3-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20231010061727.205886-1-chaoyong.he@corigine.com> References: <20231010061727.205886-1-chaoyong.he@corigine.com> X-ClientProxiedBy: PH7PR17CA0023.namprd17.prod.outlook.com (2603:10b6:510:324::24) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM8PR13MB5192:EE_ X-MS-Office365-Filtering-Correlation-Id: 254f8217-4448-4f99-5be0-08dbc958a0d4 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: I1vMFotZrzNHkJyxP8vWc+UPAkzYSTTcZHkbW540qP/c1p2950yCwa0vCcSzqjjKlpwkulYsHUJfc50QFwqzRXj66MVjLFgiTiYzfSyFnLVZlcJPN95FMIhW0PnsvdmKnCvBRr18smyk5en3cyvNyWsJrkQ1s6P3ujXmyLHnmwWtGMawh9lOrEgknWkGzmrAIuec8C1i9sswqf1PGMsITdCvMcFamOmvJUNYpgv0UHdOsRmsE1iffst6CKqGktBE4QClxNOmqI76u6tiL0ASEfOp0V1g1IUT6Jb0wgSTuXTwa3gDASkGQSU5wttG+sxHmgzH0FDv6soU3drBPI+xxN8JKwZV/NnS/aL4vhy+aGyNTwOhZnMJ0C6/Azs5MB07EB2/ai+M7Co420YkftBMU+M64IRJfZEuLkQs75QtwXpbfUDTxvKocoRrEbL2abfTYF/vtq+C0k2iIVCvW4GWC7v/C6PbOyI0bA55bOU3Otyi5mNBYHrEM1jY7vfRLyadwQfA1iLoeE95RyRzxRW2z7J6sdncPmwd2MVgKULNDqSHifivtjIZQyw/EAigXFV12aVmUBSt/NfiWeReLa3hgK2UDDI0TvB1dfi7LiMtF09G/EH1gAShMPFkSj5cToOsMsPKiOp62jArLl0WjwqxGnNrTzEtbPrNvXne2ST6b5Q= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(396003)(39830400003)(136003)(376002)(346002)(366004)(230922051799003)(451199024)(64100799003)(186009)(1800799009)(5660300002)(450100002)(41300700001)(8936002)(8676002)(4326008)(44832011)(86362001)(2906002)(4744005)(52116002)(6506007)(6512007)(6666004)(478600001)(107886003)(83380400001)(36756003)(1076003)(2616005)(26005)(316002)(6916009)(66946007)(66476007)(66556008)(38350700002)(38100700002)(54906003)(6486002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: hh74oGILtISbtblSd/YNi7BRvnPuvWDOKr4Naomw1WvUQqsBMhrnFMv0vEjpI3IpgGoYOSZ5wUv97iFlTRORuQRnUR7f6jAMRM09/U6KGQQ/prfbTh7VgJWUXWvY5gvsc3pV+uGKp1ecca2W+nHb44D6PJ2YKAumazN+LWWdvG8tKREQEIdUPvuqdetJ796a2GaTJabMxiZ+ab6pY2xbAtLxyfKAmpVDLeZoealzaucketH8+LQ6lfOT4aC+At4Hjn+sOAd2wbZsUUs22rJH9jVFFiMds5nhRDgEJEWsQTUsXalaJf3/UCRsU0ntI/ZxEwoX9sNi3uwSjsCPFhQMifLEW3q7MXaNKeBMaWYYXfK+jCFtDLlumVDCVPhEY+2sE4oKiRxbiQQ6WrU/KUMrV8B4cOv1tZBWjZoHHOJ6u+RC2ncGV/8PIahlDWK4pEMa78IcnUxdhT8RnsPaz0hyd6H0RYuZoO52nDSIkNCkinI9+ZyXk/vn46AkLALKqUMryqc+af8Gol7ILnOJ3NEUspNlzdlAvZG+ruQHswHD9br65lchYZ2qVt1ZA4g/10JEoyUERsjAO41Ffzp46vQlM2z1sGk4cHb/Dp7zz29nT2ThCNfU9n0j0rFoqC686r61KpRexjwn8jQqx/PGs7b2otkzb2HqThyD2osoyykl4OOfnr9F2ZeASG3hwbW8+99CcJZs98EioMSVxebv7BnxNt/6zmjqrAMmczXwmJuFE57h6TXeHpKhCL0i0nBWQ0limtGE5lCZ/2hbYCuVThScD1iYto8I1UKVszh9Gt4QgdkE74PUp0DHDxQqSx9hm+PVLOZd7sJwy1oQFTpQUfgnU3jOiiTDvF2zVmY8JWlnus7DpKjN93OLugXFdYSR17sdZenw6/7g6JMDQeolPLUCa0hQ+hZnt/W3Lmmzfho+/qZshVHsuFjSyJ34aEysJjrVatE6ix+mHENZ7/HoTT4uP97cIcDBhwqLnKSl+S3IXZD+BEZuef/kmSucyEbfcskZVD2+Z/pm7K0le1HqA1g1uUZpg23nbXk/T6qfEXpgthZOEKOAaGm9sxnzQXsTzB5njEH4gQlx8RWZuSim/IDGt8Pb545c2aUdibGb9yDB/2nbaUUadgeiNNbPeu9Dt17AZ8O7j09MzkWSVW7H81lvHR0Q8oLRnEt5JCwDg5BJI8OzcZCxzwDPJP7L/9+mpVYSMy5QhOK+dP6TBqefGNWWXXmLZxQTQZE3YkF6iTeCAwbWgsXlwq7btqFwRnSG48foWg0rCbFtd5RE3PSDo++PQaYqs7fzmkrD7MbtjcTdEqLgnhBs9KId6AnauGSs/HpBVxGN22HEMa3Ne/zqlsJgZJQS74lMIRZJwe3zpm/Hrak5pNx9hT8IfK0VKD+EJQH0IDptsT5ZmykuCmL2DRIl7nLv+Kps3lq89OfCw09YjR0GkB8IZhpGVg1pIEfAjimW5XLE0cWSx3qpTbcHHS/J/+SB+gAyXzDoXlbWFzZkNXqrxR4GGFGtyCDX2a6G+y8h/zMA0G5gZI5RC+wWuxVzZC1C/fBB2dN339uVAseh0Thg2AfNbAsT/EpAjnmNhdW5J5Y9W26jo9Cech29PIVbYw== X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 254f8217-4448-4f99-5be0-08dbc958a0d4 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Oct 2023 06:17:49.8555 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: pkbYaT6DCKEimdSQkyhsGWpL+bi14Mmu+GQFb8kyU0gUriGGUjORqqVdjqqLYXQ+8H1ARnEnayO/KDdoqr7qfvZa54u/jbgChn6NB7PDmMs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8PR13MB5192 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 From: Long Wu CI found that overrunning array of 8 bytes at byte offset 8 by dereferencing pointer. Coverity issue: 403098 Fixes: 9641a2d ("net/nfp: refactor rtsym module") Cc: chaoyong.he@corigine.com Cc: stable@dpdk.org Signed-off-by: Long Wu Reviewed-by: Chaoyong He Reviewed-by: Peng Zhang --- drivers/net/nfp/nfpcore/nfp_rtsym.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.c b/drivers/net/nfp/nfpcore/nfp_rtsym.c index 022753f98a..5cefbace96 100644 --- a/drivers/net/nfp/nfpcore/nfp_rtsym.c +++ b/drivers/net/nfp/nfpcore/nfp_rtsym.c @@ -337,7 +337,7 @@ nfp_rtsym_read_real(struct nfp_cpp *cpp, size_t length = len; uint64_t sym_size = nfp_rtsym_size(sym); - if (offset > sym_size) { + if (offset >= sym_size) { PMD_DRV_LOG(ERR, "rtsym '%s' read out of bounds", sym->name); return -ENXIO; }