linux-kernel - RE: [PATCH] gfs2/inode.c: local functions should be static

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]

Message-ID: <1317062248.2744.28.camel@menhir>
Date:	Mon, 26 Sep 2011 19:37:28 +0100
From:	Steven Whitehouse <swhiteho@...hat.com>
To:	H Hartley Sweeten <hartleys@...ionengravers.com>
Cc:	Linux Kernel <linux-kernel@...r.kernel.org>,
	"cluster-devel@...hat.com" <cluster-devel@...hat.com>
Subject: RE: [PATCH] gfs2/inode.c: local functions should be static

Hi,

On Mon, 2011-09-26 at 11:54 -0500, H Hartley Sweeten wrote:
> On Monday, September 26, 2011 1:28 AM, Steven Whitehouse wrote:
> > Hi,
> >
> > On Fri, 2011-09-23 at 15:51 -0700, H Hartley Sweeten wrote:
> >> Quiets the sparse noise:
> >> 
> >> warning: symbol 'gfs2_initxattrs' was not declared. Should it be static?
> >> 
> > Where did you find this function? It is not in the upstream gfs2 code...
> 
> linux-next next-20110831
> 
> This was the last update I pulled before git.kernel.org went down.
> 
> It looks like it was added by:
> 
> commit 9d8f13ba3f4833219e50767b022b82cd0da930eb
> Author: Mimi Zohar <zohar@...ux.vnet.ibm.com>
> Date:   Mon Jun 6 15:29:25 2011 -0400
> 
>     security: new security_inode_init_security API adds function callback
> 
> If this has been removed since then please ignore.
> 
> Thanks,
> Hartley

I think it is a question of it not yet being added. You should send the
patch to Mimi so it is included in the correct patch series. Otherwise
remind me when the code is applies against is merged and I'll look at it
again then,

Steve.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/