netdev - Re: [PATCH] New Chapter on CodingStyle .

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170815092030.7ec82ace@xeon-e3>
Date:   Tue, 15 Aug 2017 09:20:30 -0700
From:   Stephen Hemminger <stephen@...workplumber.org>
To:     David Laight <David.Laight@...LAB.COM>
Cc:     'Jonathan Corbet' <corbet@....net>,
        Corcodel Marian <asd@...ian1000.go.ro>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Realtek linux nic maintainers" <nic_swsd@...ltek.com>
Subject: Re: [PATCH]  New Chapter on CodingStyle .

On Tue, 15 Aug 2017 10:42:39 +0000
David Laight <David.Laight@...LAB.COM> wrote:

> From: Jonathan Corbet
> > Sent: 12 August 2017 15:55  
> ...
> > > +		Chapter 20: Put values on initialisers without exception
> > > +
> > > +When declaring variables on functions must put values:  
> > 
> > Thanks for sending a patch for the kernel's documentation.
> > Unfortunately, I can't accept this patch for a couple of reasons:  
> ...
> > - The coding style document is there to describe the community's
> >   standards for kernel code.  It is *not* a mechanism for imposing new
> >   standards.  If you really think that the kernel community should adopt
> >   this rule, you will need to argue for it on the mailing lists.  I will
> >   say, though, that I do not expect that this effort would be successful.  
> 
> I'd even go as far as suggesting almost the opposite.
> Declarations should only have initialisers if the value is constant.

Yup. This new rule sound like something taught to people in coding schools.
But initializing everything defeats the compiler detection of uninitialized variables
which is more useful for catching errors.