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
| ||
|
Date: Fri, 4 Aug 2017 09:51:15 -0700 From: Tyrel Datwyler <tyreld@...ux.vnet.ibm.com> To: SF Markus Elfring <elfring@...rs.sourceforge.net>, linuxppc-dev@...ts.ozlabs.org, Benjamin Herrenschmidt <benh@...nel.crashing.org>, Jan Kara <jack@...e.cz>, Jens Axboe <axboe@...com>, Dan Williams <dan.j.williams@...el.com>, Michael Ellerman <mpe@...erman.id.au>, Paul Mackerras <paulus@...ba.org> Cc: kernel-janitors@...r.kernel.org, LKML <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 4/4] axonram: Delete an unnecessary variable initialisation in axon_ram_probe() On 08/03/2017 12:17 PM, SF Markus Elfring wrote: > From: Markus Elfring <elfring@...rs.sourceforge.net> > Date: Thu, 3 Aug 2017 20:34:00 +0200 > > The local variable "rc" will eventually be set only to an error code. > Thus omit the explicit initialisation at the beginning. > > Signed-off-by: Markus Elfring <elfring@...rs.sourceforge.net> > --- > arch/powerpc/sysdev/axonram.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/powerpc/sysdev/axonram.c b/arch/powerpc/sysdev/axonram.c > index 93cc902350db..5677f3371e30 100644 > --- a/arch/powerpc/sysdev/axonram.c > +++ b/arch/powerpc/sysdev/axonram.c > @@ -184,7 +184,6 @@ static int axon_ram_probe(struct platform_device *device) > static int axon_ram_bank_id = -1; > struct axon_ram_bank *bank; > struct resource resource; > - int rc = 0; You've completely removed the decleration of "rc" instead of removing the "= 0" initialization. I would expect a compilation test to have turned up an undeclared use error for "rc". -Tyrel > > axon_ram_bank_id++; >
Powered by blists - more mailing lists