linux-kernel - Re: [PATCH] ARM: ptrace: Include linux/audit.h to fix build errors

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: <CACkDPNdX0L0mXTo5Net+A2jKsxws8RWhNocNKOzUqQQptRNqRA@mail.gmail.com>
Date:	Wed, 15 Feb 2012 11:14:36 -0500
From:	Nathaniel Husted <nhusted@...il.com>
To:	Russell King - ARM Linux <linux@....linux.org.uk>
Cc:	Eric Paris <eparis@...hat.com>, Will Deacon <will.deacon@....com>,
	Axel Lin <axel.lin@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Dave Martin <dave.martin@...aro.org>,
	Nicolas Pitre <nicolas.pitre@...aro.org>,
	Ingo Molnar <mingo@...e.hu>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] ARM: ptrace: Include linux/audit.h to fix build errors

The code in place regarding the endian-ness was put into place long
before my patch and I've no clue the original reasoning behind it.
Thus I'd defer the answer to Eric or other powers that be.

Cheers,
Nathaniel

On Wed, Feb 15, 2012 at 11:10 AM, Russell King - ARM Linux
<linux@....linux.org.uk> wrote:
> On Wed, Feb 15, 2012 at 10:10:58AM -0500, Eric Paris wrote:
>> On Wed, 2012-02-15 at 10:18 +0000, Russell King - ARM Linux wrote:
>>
>> > So, this gives us two options: either we revert the damned thing and start
>> > over with it (and hopefully audit folk will start co-operating with ARM
>> > folk) or we try to fix up their mess (and potentially they learn they can
>> > dump crap on us.)
>>
>> It must be a conspiracy to screw with the ARM people!  Your jobs were
>> obviously too easy to begin with.  Rahrrrrr!  Down with ARM!
>>
>> No seriously.  I am the audit folks.  You got us all right here in this
>> one e-mail.  You're right, I should have paid more attention to who was
>> in the cc list.  If you want to rewrite it, please feel free!  If you
>> want to push fixes yourself, please feel free!  If you want me to push
>> fixes, please just let me know what is broken and I'll do my best.
>>
>> Sorry.  I just got the patch and was contacted by a number of people off
>> list asking when it was going to go in because they wanted it for their
>> project.  I obviously should have known who the ARM maintainers that
>> would have looked at a patch for the audit system was and should have
>> made sure they commented.  Sorry!  I'll be sure to hit the ARM list if
>> another patch goes by....
>
> And... what about this endian-ness issue?
>
> Frustrated.
>
> I've tried discussing that issue with Nathaniel and haven't got anywhere
> with it (other than, seemingly, it's above his pay grade to know the
> answer.)
>
> Right, I've committed a revert of it to my tree.  When there's more
> interest in the feature, it can be revisited with proper review and
> proper testing of it.
--
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/