From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 666EDC433FE for ; Sun, 9 Oct 2022 15:25:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229797AbiJIPZY (ORCPT ); Sun, 9 Oct 2022 11:25:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59456 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230115AbiJIPZU (ORCPT ); Sun, 9 Oct 2022 11:25:20 -0400 Received: from mail-qt1-x829.google.com (mail-qt1-x829.google.com [IPv6:2607:f8b0:4864:20::829]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 206B31D66D for ; Sun, 9 Oct 2022 08:25:19 -0700 (PDT) Received: by mail-qt1-x829.google.com with SMTP id c23so1341597qtw.8 for ; Sun, 09 Oct 2022 08:25:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=zMWBP42bcqYR/9/IcRRsvCkgoPBmDNpE4tq769Q64oI=; b=hzPt6d3JeW0Footcf5s588X/R44BO81hmMDQLXWaFZJY5KbRKNCumivhg/rE96IjH2 Kw0XoDHKhAOlIOhvgoIrD7k8SILH88ecSk16n/o8Wg6aV25QILm+OGF1vXYxAT3e77FQ Ew7GlZ/QGNq7w36LYmKwZFNscIEGwPkYA1qgHpBf1koWgvgZeM03ZIm1rqptWbZIAsE/ X1tYIXazKHgCvi13Dvhr3XD2f/0oLPr2loqEQJOBzPUwJ1svmyqKy6wL1eE8l80zrIPj lN6u8JkgPlyu1zkDaYukKKjsJ1xvIbxYYq+bQhQ91CFCjJmBB5gE+lKOgF7WYjGjPIDX 0Atg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=zMWBP42bcqYR/9/IcRRsvCkgoPBmDNpE4tq769Q64oI=; b=mr4Kii0ihiBMpOSnS1fHmJ5M0i9rZa9ppx2+ELGX62yFG5/eVGZMg+XGPcLlgES0j2 E10vlUsHdKLppSaU+6PPwn4Sxeps/8UoubLalJJdjmSVZAIC6lak2iarBvqMUe1n+Och FF8SAk0PnegmH9qBthxY146E9VeeMrJTmhr6Y7OUmd/sVL7QkWeqHF5oI4i8XkESfam9 IKCFnrFpq1OGC5gkbPWgODyhQSBw91Udd+qyh99hacrBi5U30dzCw7jIxv6fs8NM5KLL wHRlZzoXoLeVW1I91DHLAumgWEXulcU8m2E6nP0U4p4vmAz9YyHGzFxshg2xzAbmbBkY nRxQ== X-Gm-Message-State: ACrzQf2erh+D6ozkNlYUVQmwf36I3g8HXKfrt/Swqw42ib6OGSLCrGwL cfOo9/eK7fdyM0fEtfi7ap4vQQ== X-Google-Smtp-Source: AMsMyM6WOzEKoFOz4nRaUQLwSDLVPIGwcjJ8aEcuHay/BBUrycXh6QCMWv6+memfJTEi64NCDRE5Tw== X-Received: by 2002:ac8:7e94:0:b0:38e:2bea:9dd9 with SMTP id w20-20020ac87e94000000b0038e2bea9dd9mr11825064qtj.67.1665329118301; Sun, 09 Oct 2022 08:25:18 -0700 (PDT) Received: from [192.168.1.57] (cpe-72-225-192-120.nyc.res.rr.com. [72.225.192.120]) by smtp.gmail.com with ESMTPSA id fp25-20020a05622a509900b0034305a91aaesm6549834qtb.83.2022.10.09.08.25.16 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 09 Oct 2022 08:25:17 -0700 (PDT) Message-ID: Date: Sun, 9 Oct 2022 17:23:06 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.3.1 Subject: Re: [PATCH 5.10 1/1] rpmsg: qcom: glink: replace strncpy() with strscpy_pad() Content-Language: en-US To: David Laight , 'Greg Kroah-Hartman' , Andrew Chernyakov Cc: Andy Gross , Bjorn Andersson , Konrad Dybcio , Mathieu Poirier , "linux-arm-msm@vger.kernel.org" , "linux-remoteproc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" , "lvc-project@linuxtesting.org" , Stephen Boyd , Bjorn Andersson References: <20221007132931.123755-1-acherniakov@astralinux.ru> <20221007132931.123755-2-acherniakov@astralinux.ru> <36f776cbc16f4e988d96b7bcb77cd559@AcuMS.aculab.com> From: Krzysztof Kozlowski In-Reply-To: <36f776cbc16f4e988d96b7bcb77cd559@AcuMS.aculab.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/10/2022 23:11, David Laight wrote: >>> --- >>> drivers/rpmsg/qcom_glink_native.c | 2 +- >>> drivers/rpmsg/qcom_smd.c | 4 ++-- >>> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> Why just this specific kernel branch? We can't add patches to an older >> tree and have someone upgrade to a newer one and hit the same issue. >> >> So please provide backports for all active versions. In this case that >> would be 5.15.y and 5.19.y. > > If it is only fixing a compile warning is it even stable material? > The generic commit message doesn't say whether the old code was > actually right or wrong. > > At least one of these 'replace strncpy()' changes was definitely > broken (the copy needed to be equivalent to memcpy()). > > So applying ANY of them to stable unless they actually fix > a real bug seems dubious. Except the warning from GCC, there was no bug to fix. The warning is about discouraged and risky practice, but no actual real risk was identified, so for me it matches stable rules poorly. It's basically backporting to silence automated code checkers... Best regards, Krzysztof