Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Robert Richter <rrichter@marvell.com>
To:     Borislav Petkov <bp@alien8.de>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Tony Luck <tony.luck@intel.com>
CC:     James Morse <james.morse@arm.com>,
        Robert Richter <rrichter@marvell.com>,
        "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Joe Perches <joe@perches.com>
Subject: [PATCH v2 05/20] EDAC, mc: Remove needless zero string termination
Thread-Topic: [PATCH v2 05/20] EDAC, mc: Remove needless zero string
 termination
Thread-Index: AQHVlIU0JPYgrRN4PUabZYpzEbCP9Q==
Date:   Wed, 6 Nov 2019 09:33:11 +0000
Message-ID: <20191106093239.25517-6-rrichter@marvell.com>
References: <20191106093239.25517-1-rrichter@marvell.com>
In-Reply-To: <20191106093239.25517-1-rrichter@marvell.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-mailer: git-send-email 2.20.1
x-originating-ip: [31.208.96.227]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: a68bd89a-4dd1-4f50-022b-08d7629c573e
x-ms-traffictypediagnostic: MN2PR18MB3165:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <MN2PR18MB316549D603B073C6B4361030D9790@MN2PR18MB3165.namprd18.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:5797;
x-forefront-prvs: 02135EB356
x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(4636009)(366004)(396003)(376002)(39860400002)(346002)(136003)(199004)(189003)(52116002)(305945005)(6486002)(4744005)(76176011)(71190400001)(71200400001)(256004)(11346002)(446003)(7736002)(14444005)(6436002)(4326008)(6512007)(66446008)(66556008)(66476007)(66946007)(64756008)(5660300002)(1076003)(476003)(6116002)(3846002)(110136005)(54906003)(36756003)(2906002)(8676002)(99286004)(81166006)(81156014)(316002)(8936002)(50226002)(486006)(6506007)(386003)(102836004)(66066001)(2616005)(86362001)(478600001)(25786009)(186003)(14454004)(26005);DIR:OUT;SFP:1101;SCL:1;SRVR:MN2PR18MB3165;H:MN2PR18MB3408.namprd18.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;A:1;MX:1;
received-spf: None (protection.outlook.com: marvell.com does not designate
 permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: BqkMX2APnI8EuNmTE2Jj7bw8WoVzcKyBWLyVjwNOJ6Y2ZgvJZXRnZNfyz8OrzIzdJ8aPqAIid3xrvNGU2mFnl/uuwt803uBkLjP/3qZsJ4cOU/w8Z1nSrBzBSpjPqbvaq/S06eO54Q0Ck4BLoKix8zF6sPMxau/6FUKZPgzxW0HkeuI7+3jRawtTFEmzFyh7D9OABSMe3f2iDQjzUcSVpUAMFgyA9sgQUipXAkYK3Lg2B+/9bBA60C5MWw1fP912l1wvXpznHTGU/mh9DRNX1w31ZVmonfdLfd7FbRhGZhitTwxR2nM2j7l4D/jk3LgyL18b1YKX72ijD0GozgM47MV/LUvexgeIcyFFrPwyuRO+FeBDrz2BvZgLCCBpI+p0kNg2U8xdKYjBf0YGiG7j9dMhKQHvpiuXRLIc0gwB8MhtkJL61uThGHBG0t5WIPvI
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: a68bd89a-4dd1-4f50-022b-08d7629c573e
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Nov 2019 09:33:11.8174
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: oa59cp5ACBPTJsweWoQsY0ROCM0S65qIE3V2Nd/ZEwUKxVWCqGeV8wFchjeg0aVeluikZWSrLJmCgL7aCZQyLA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR18MB3165
X-OriginatorOrg: marvell.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.95,18.0.572
 definitions=2019-11-06_02:2019-11-06,2019-11-06 signatures=0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Since this is a string already and strlen() has been used to advance
the pointer, the end of the buffer is already zero terminated. Remove
the needless zero string termination.

Suggested-by: Joe Perches <joe@perches.com>
Signed-off-by: Robert Richter <rrichter@marvell.com>
---
 drivers/edac/edac_mc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c
index f2cbca77bc50..8bfe76d1bdf1 100644
--- a/drivers/edac/edac_mc.c
+++ b/drivers/edac/edac_mc.c
@@ -1183,7 +1183,6 @@ void edac_mc_handle_error(const enum hw_event_mc_err_=
type type,
 			}
 			strcpy(p, dimm->label);
 			p +=3D strlen(p);
-			*p =3D '\0';
=20
 			/*
 			 * get csrow/channel of the DIMM, in order to allow
--=20
2.20.1