netdev - Re: [PATCH net-next 1/1] Remove the redundant initialization statement.

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] [day] [month] [year] [list]

Date:	Tue, 14 Jun 2016 22:28:23 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	sudarsana.kalluru@...gic.com
Cc:	netdev@...r.kernel.org, Yuval.Mintz@...gic.com
Subject: Re: [PATCH net-next 1/1] Remove the redundant initialization
 statement.

From: Sudarsana Reddy Kalluru <sudarsana.kalluru@...gic.com>
Date: Mon, 13 Jun 2016 05:01:10 -0400

> The callback ".getcbx" is being initialized twice. This was introduced by
> the commit 0fdeb72aa6c9 ("qede: Add dcbnl support."). The patch removes the
> redundant initialization statement.
> 
> Reported-by: Julia Lawall <julia.lawall@...6.fr> 
> Signed-off-by: Yuval Mintz <Yuval.Mintz@...gic.com>
> Signed-off-by: Sudarsana Reddy Kalluru <sudarsana.kalluru@...gic.com>

This subject line is not properly formed.

After "[PATCH ...] " you must provide a proper subsystem prefix specifier
followed by a colon character, and a SPACE.

Otherwise nobody reading the GIT shortlog has any idea where in the kernel
your change is taking place.

An appropriate subsystem prefix for this patch woult be "qede: ".