Message ID | ZUr67-KKyaSDv_vk@alg-gentoo |
---|---|
State | Accepted |
Commit | a40f35f0691f862d527ed75329773a24369a5b7e |
Delegated to: | Tom Rini |
Headers | show
Return-Path: <u-boot-bounces@lists.denx.de> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=matrox0.onmicrosoft.com header.i=@matrox0.onmicrosoft.com header.a=rsa-sha256 header.s=selector1-matrox0-onmicrosoft-com header.b=DkyTE8g5; dkim-atps=neutral Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org) Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4SQB5H2Wy3z1yRF for <incoming@patchwork.ozlabs.org>; Wed, 8 Nov 2023 14:53:21 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id A0FB487668; Wed, 8 Nov 2023 04:53:09 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=matrox.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=matrox0.onmicrosoft.com header.i=@matrox0.onmicrosoft.com header.b="DkyTE8g5"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 3D73F87689; Wed, 8 Nov 2023 04:05:25 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.2 Received: from CAN01-YT3-obe.outbound.protection.outlook.com (mail-yt3can01on20600.outbound.protection.outlook.com [IPv6:2a01:111:f403:241c::600]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 3EFAD87661 for <u-boot@lists.denx.de>; Wed, 8 Nov 2023 04:05:23 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=matrox.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=agendin@matrox.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jh1h6IJuWVK1458XkXRlmLujd8JljJeAttY+ayNSI2DeJldn3EGFQxdgQ5F8CVUyaETd17GKSDqMunDq1U+6g9KYMtuHMN90YE4szGU5iFlaFaqTCADffPSODiFpKOP8diBldZp9ifX5qLZgNBtxXQl6bfLu15WTTlVPCKrsS3bc+Y/in6ebsAayHPBzv/kjOND3nmu8FKpXxlo5DU7lL2uMo2eyfX6p9oFANkfv1lZz5Uu6fZr6UgfzC6gZt6uuhpQCU2EgHaWuPGOvnSu3UUGxA+1NJE9cgM9bgZrHX7XCD7sko4KR3fs0FHOhefBpSbw8L5rw3G1NNFW3egRQKw== 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=mdNq9iagGxCDzhfDPNW2msCkosVAqzB7Mh0DLlEeobU=; b=C0Z17HDsL23mhjk25enP/CDRB1xEByMpClKjB8ut7eZ1W1LBj3dRnsdO+ounzYfne1msCuwh3PT1H6cnN3pgmKsX573BuHQNPwo3sTXbeUnRawikk5/UP366pn4ipJUW+s9EOG1F8MvDj5kHjAKHVBTuzMCzP+m4mQeiocT+sPVtnxRyjnL3RCIW4/rupSNF2DpGoY9GWUZDQzWx1SVuS27dhMadgr7tTi5/fR/AgKftNxJ51bXcg9VbT5Q35eTx4IdQLTpxegmgAzx8RpUVxNBWp68rw1NMdP4wBTg1BgQhH2w4GoUkfXNPyts00IEhp0I4VRFb0CJGQadDz/tQQA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=matrox.com; dmarc=pass action=none header.from=matrox.com; dkim=pass header.d=matrox.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=matrox0.onmicrosoft.com; s=selector1-matrox0-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=mdNq9iagGxCDzhfDPNW2msCkosVAqzB7Mh0DLlEeobU=; b=DkyTE8g5f47oV8ZQsFec1GipywJG2cnhtePdnY5nkUdqb7NII7oRkgvUMNT2Ih1vgG6z8EDa63VaSOdbujpHaT7AhjAHgL5Yv1prNNrT77oSqRyjyJPbsb/hMwFm4/R47IwaCjiiNnDXTFhf37sfejtaj0/hultvLkoOosc3SHA= Received: from YT3PR01MB5626.CANPRD01.PROD.OUTLOOK.COM (2603:10b6:b01:60::18) by YT1PR01MB8329.CANPRD01.PROD.OUTLOOK.COM (2603:10b6:b01:c0::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6977.18; Wed, 8 Nov 2023 03:05:20 +0000 Received: from YT3PR01MB5626.CANPRD01.PROD.OUTLOOK.COM ([fe80::a17d:3de3:e313:a672]) by YT3PR01MB5626.CANPRD01.PROD.OUTLOOK.COM ([fe80::a17d:3de3:e313:a672%4]) with mapi id 15.20.6954.028; Wed, 8 Nov 2023 03:05:19 +0000 From: Alexander Gendin <agendin@matrox.com> To: "u-boot@lists.denx.de" <u-boot@lists.denx.de> CC: Tom Rini <trini@konsulko.com> Subject: [PATCH] test: cmd: mbr: Remove unreachable code Thread-Topic: [PATCH] test: cmd: mbr: Remove unreachable code Thread-Index: AQHaEfBoLgRdAisnHEe1wG0NTM0Mww== Date: Wed, 8 Nov 2023 03:05:19 +0000 Message-ID: <ZUr67-KKyaSDv_vk@alg-gentoo> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=matrox.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: YT3PR01MB5626:EE_|YT1PR01MB8329:EE_ x-ms-office365-filtering-correlation-id: 54386024-33b0-4b72-b548-08dbe0078ab6 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: je3UcRm+8XQATnWNKoFoKYV9hj18j6wcGPyFNt8M7eDSKOJYhnHN/jaCZxaFmu8IrQkfqsZmGHTls/Ha1Kj6AOz5VVepenGisgH7KXsFGzFf75lFv9hhgCI5jBv9CmRHXV4+N0rzrrPqeXh1hXFKR6KgcVNR3XxiwW6WCcwJB2SKpyEgjcIhsX/JZb2wUoOOJwgQT7yrFgoyPFHxsNMbn45jV8R8bfNyFll+3cNfRX3dthPxkMbS+H+4KFrNizXx/FDlsYMutjKlfFza9OyI/RFXT1gaLehDsl4KMYpo47tG2a+yfUs9o2HeXc72SMeE4dwwDU6XcJ8nuVQCAg3KR/LwGJpV8KkfcXGBKY102/vrE6e8HdG3wwh7QOucomaIb3rV7QLy3YDB4KkBg3ne/gaKHgaYKvpqHuqRwc5U0ptejNJDv0FVD75AIQFFoFJP4UpCW6B2qcZAU+28wELoxMrnLSqn6nFPPazvRmIf3rdTklJBdFdnUVkSMYR3T1lHdckFu8es90Yi7XmaZrhg3pilNAMwO1LKJtiS3SGx3QXKyu/Y/f/4aaK+KLuZa+SGVwXPbJE3K+typQPAUWycQTtWjks2GWLuYxYm826dcGY/m1sjBBjE+JPwQW2FoIrP x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:YT3PR01MB5626.CANPRD01.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230031)(7916004)(39850400004)(396003)(366004)(136003)(346002)(376002)(230922051799003)(451199024)(64100799003)(186009)(1800799009)(316002)(122000001)(86362001)(6916009)(76116006)(91956017)(64756008)(66946007)(5660300002)(66446008)(66476007)(66556008)(478600001)(8676002)(4744005)(8936002)(6486002)(4326008)(2906002)(71200400001)(38100700002)(6506007)(9686003)(6512007)(33716001)(38070700009)(83380400001)(26005)(41300700001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?q?7n1c7j/Fqme18R/yeCn7wOqoM/o3?= =?utf-8?q?aCBnyeSyFPVXvhTNVnrdrIoy5otuQMk/Mo0RpQryXNnrMg5lE2EpmYobpAsgBUFuN?= =?utf-8?q?O0aPyP6lcpSSue8KLQqybrdvY8rvwKEviFLYJPjDL1BZ5dVylMNmzYk/Hvoz9iiUo?= =?utf-8?q?Ru7rTJoNMzZ2CtkU2FtAmcU/pRVR6hkmUNw3PEUlbZrwWvOGzkxQGyoR9U5kSfVkR?= =?utf-8?q?xCHSzjqTlrV7xtnhokGlnSj1gvcUVTN1cgeNjIHy+8zzjBpcAuOSRYKRiFq6Q+2iE?= =?utf-8?q?L+S1/e/qv24wlZW1RqbZ+LVeAMNEQ25mgD+6f6tjE12AWT+iewbwevHRTUzD2LT2/?= =?utf-8?q?1OX8KzlpT2dTVDotfxozHPtNzVmZ8oNdrim8cAXBPTgjQ20DrP40yI2aDkGSURxCe?= =?utf-8?q?2/VDZyOMpEqWDd69MxGMdu+yT54Wxi0WxjhHjSEye8dJu5YOhYkxVfcELOUcbnGg+?= =?utf-8?q?yfolswrx8LeyNVufm55AaTmRPq+e7ULYDHHls2oGhWiQdtW5CYlh5KeG5pEEizycq?= =?utf-8?q?zaTzuiG863qxj8gxmRXauJj22R8dop/uynqRH6wWwu0DxSnlbmlADtUETXnfcZykO?= =?utf-8?q?QoYdkzmgQzhMTL6mceE99kPWHySb23FDWP4+ecRrIMAgPVhTZcXUH6Tzq7l+c7+bD?= =?utf-8?q?K6bOfosNQ0vT1i1uDIljRB3dJ/ZKTohwEtS9z4bH3SEOcXBuueCMQ/gwcLkJd37av?= =?utf-8?q?76miX8aKePMzFpQDIzOJQXuLmepmnA6nXwvsRLi1pnIXq58FOl1151LMUXKEBzlE4?= =?utf-8?q?c1HLAjulNcRCyVOKVKjwpLsLsrdg58rEj7MsFjYmI3CqZzg4qopFCp2a+PHY4uB7N?= =?utf-8?q?7jpOsvmJ87bOMeCo7HQ+wsnqJ24ZpPugPPi6/iD0vh8Oukklord615J+ViLp0WeaM?= =?utf-8?q?VIlrqscPKeOmXl9EgdTSyGC+X1AlPxSszeuexfL64HXgG0SObKYeRD9rRvjmtPBVT?= =?utf-8?q?6VB3QHxWG82zXopTvV9lNaMLVUewKYgMtYBy7BwYXZbLevo7LNNqTIzDGrXKJxaW8?= =?utf-8?q?Ie/bJu/yjGFdnJRbbSMGV14x4swqP+ZjdkEcB5S+PFzVwv7y6y2NT2h0JuJMg5MBE?= =?utf-8?q?ooQiE3/Idk5B9qLutw0Wh+MGpfVzAR8Z2+4nlKPmqgbp5C1T+Nf/ytIyBksHyHIny?= =?utf-8?q?ZTio7xlbTfRSvLQ5wAzRaHVGHlMUVHDOkzV5s/4QczSMx0mesQCEdXn9CfVeWspHf?= =?utf-8?q?2yDdwG2jRbI1feYBiJswmIx6aN6dEQqb0gMdbuyFWxTMgMalJLixkCIH1O1ft3BDc?= =?utf-8?q?3A56XZtIEOE0lp+v+OsxBDR0sgnhbWAUy7mM0ux+xgZrl5iHkR6ba2ncNqhbyxJjT?= =?utf-8?q?zryo+FTL/IPz69cXWCQ4vozl6Be8Yubkbi3yxE14nAP8csmzIQcHUbB6C8kw/pPs8?= =?utf-8?q?J0UIIx9dM3tYZpXofM2tvvVjl7PW0btSyALSkjxM4mniAHLLPSKRBZeCz/9vGUwzu?= =?utf-8?q?uQWkAt0Px52Yfhf2x89KDcqP7ltRKu1HoKRgVaZxxUoAeqOafc6+Yx7DhaOEkd5mz?= =?utf-8?q?xSDyLmWHHg07?= Content-Type: text/plain; charset="utf-8" Content-ID: <940849AA38120140BF4D0F76D7485240@CANPRD01.PROD.OUTLOOK.COM> Content-Transfer-Encoding: base64 MIME-Version: 1.0 X-OriginatorOrg: matrox.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: YT3PR01MB5626.CANPRD01.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: 54386024-33b0-4b72-b548-08dbe0078ab6 X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Nov 2023 03:05:19.9671 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 09c2e324-8cf7-459c-b6b4-e23db24b9d4a X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: w90KbfnnCQBZF3AMSaMbwoAR519WQwzFLCbsJMHE7L1vrHhFsFvZIDNu8lTgCTcwohmXbW+7IeNMDH9HLlG3mw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: YT1PR01MB8329 X-Mailman-Approved-At: Wed, 08 Nov 2023 04:53:08 +0100 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion <u-boot.lists.denx.de> List-Unsubscribe: <https://lists.denx.de/options/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=unsubscribe> List-Archive: <https://lists.denx.de/pipermail/u-boot/> List-Post: <mailto:u-boot@lists.denx.de> List-Help: <mailto:u-boot-request@lists.denx.de?subject=help> List-Subscribe: <https://lists.denx.de/listinfo/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=subscribe> Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de> X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean |
Series |
test: cmd: mbr: Remove unreachable code
|
expand
|
On Wed, Nov 08, 2023 at 03:05:19AM +0000, Alexander Gendin wrote: > Fix Coverity (CID 467404): Control flow issues (DEADCODE). > Fix code indentation. > > Reported-by: Coverity (CID 467404) > Signed-off-by: Alexander Gendin <agendin@matrox.com> Applied to u-boot/master, thanks!
diff --git a/test/cmd/mbr.c b/test/cmd/mbr.c index 5d7402154d..46b78e706c 100644 --- a/test/cmd/mbr.c +++ b/test/cmd/mbr.c @@ -205,16 +205,14 @@ static unsigned build_mbr_parts(char *buf, size_t buf_size, unsigned num_parts) bytes_remaining -= cur_str_size; } - else if (num_parts == 5) { - cur_str_size = sizeof(mbr_parts_p5); - if (cur_str_size + 1 > bytes_remaining) - return 1; - strcat(cur_buf, mbr_parts_p5); - bytes_remaining -= cur_str_size; + else if (num_parts == 5) { + cur_str_size = sizeof(mbr_parts_p5); + if (cur_str_size + 1 > bytes_remaining) + return 1; + strcat(cur_buf, mbr_parts_p5); + bytes_remaining -= cur_str_size; - } - else if (num_parts > 5) - return 1; + } } } }
Fix Coverity (CID 467404): Control flow issues (DEADCODE). Fix code indentation. Reported-by: Coverity (CID 467404) Signed-off-by: Alexander Gendin <agendin@matrox.com> --- test/cmd/mbr.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-)