Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp511764imm; Fri, 13 Jul 2018 01:14:14 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfHUJ+nEmN/tDP3iNmtCVZzvX3LIkKeV2JoC015tPE53XA/3LU7S/o8/2zv/a66hM8qRrXv X-Received: by 2002:a17:902:9f81:: with SMTP id g1-v6mr5332215plq.304.1531469654918; Fri, 13 Jul 2018 01:14:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531469654; cv=none; d=google.com; s=arc-20160816; b=s79nEfS+5Kj28gY+phZe8UsxBItsry/gV09tUEPFKKw9jAC70b4r79qaiQV/Cl1Ubr clEjAIN6hg/Fq4LlcPbAGTiAEFWuvcIAg+IwwMTqOEbbkD8otgZNFYwG5eP3PNMJZFix 7jeVFDp4vDiZCHIiOB4F8EaYgOE0SELPO2m0MevEaHxcUmelysvt+OZotQOfCR/luoMR wjLqNxpPR3BXdNDldCqum8yOJUCh+ilH1712hUfGjtha0CluuQnZjequF+GFUj3NTbO8 n4dDjl1lBjp3S+JqWNQ+hvCVVkW/wL8ijsurWzHIE2CwUVGD7+yrxU5k5Au2xRIsCPN+ RX4w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=YrEN0x7LrUcY24h8BhRyxbUGxfoeovHFkIRgQ/84GK0=; b=qbCv0fAmqtsGMMvKTEpY86qvJKDJlJbkwYaPG/Ch05x9VeqtnAP/SPJ3XOpjtoHGBO /6wbkNk5QCORA9CdHdx8dPtg5MsSADIGCHzLXGQUT0nxi+nFxrfVoFAnxT/sfmDhzKhW wYBmJoFL9dQetCrsSTgcIg+l5mm4C6Ehdwo+oorBlCC+UArFRaEWYlj/a0MkE/0NFJmr xElVkDk4iEO2oXymvzwqpRC2i3fYoMUhSJ1UZJAd41kUny5FSfxlN3hM27Pi9s1Rtu2z juOXySngbYS5BtbB5/QG1hE6xA5jcv3DyMV7JZh1N3cc1PuzIZ9rzwFm5vnmoJmA1nfZ c00w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d8-v6si15353794pgn.382.2018.07.13.01.13.59; Fri, 13 Jul 2018 01:14:14 -0700 (PDT) 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; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2387843AbeGMI06 (ORCPT + 99 others); Fri, 13 Jul 2018 04:26:58 -0400 Received: from nautica.notk.org ([91.121.71.147]:37613 "EHLO nautica.notk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731968AbeGMI06 (ORCPT ); Fri, 13 Jul 2018 04:26:58 -0400 Received: by nautica.notk.org (Postfix, from userid 1001) id 054E5C01A; Fri, 13 Jul 2018 10:13:24 +0200 (CEST) Date: Fri, 13 Jul 2018 10:13:08 +0200 From: Dominique Martinet To: Arend van Spriel Cc: Greg Kroah-Hartman , Kalle Valo , "David S. Miller" , Thomas Gleixner , Kate Stewart , Philippe Ombredanne , Joe Perches , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 11/18] wireless/ti: change strncpy+truncation to strlcpy Message-ID: <20180713081308.GC9638@nautica> References: <1531444483-17338-1-git-send-email-asmadeus@codewreck.org> <1531445149-19607-1-git-send-email-asmadeus@codewreck.org> <20180713073810.GA31984@kroah.com> <5B485917.7090102@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <5B485917.7090102@broadcom.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Arend van Spriel wrote on Fri, Jul 13, 2018: > The patch adding that script contains a good motivation, but I would want to > see that in commit message of every patch or at least the gist of > it. In retrospect, I definitely agree - I was happy I got coccinelle to work and a bit too tired to make rationale decisions when I sent the serie as it's not a kind of thing I'm used to. For the patch you ack'd, in particular, there would be no gcc warning in the first place because the source string's size is not known at compile time and for some reason gcc does not mind silent truncation in that case, so the usefulnes of the patch is fairly limited in the first place (it's possibly simpler/good to aim for consistency but that's about it). I however didn't take the time to make that analysis for all the patches. > especially as that script is not in the kernel tree yet. I did think about that, but wasn't sure what was appropriate in this case. I now think it would have been better to save everyone a dozen of mails and wait for the coccinelle patch to land first; but it's a bit late for regret :) I'll only catter after the coccinelle script until it lands, so if anyone is inclined to take one of the rest as they are, great, but otherwise feel free to ignore them for now. (In particular, this very patch should not remove the first comment here, as pointed out by Himanshu Jha in reply to the first patch) Thanks for taking the time to give feedback, -- Dominique Martinet