linux-kernel - Re: [GIT PULL] MMC updates for 2.6.26

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]

Date:	Thu, 17 Jul 2008 09:38:10 +0300
From:	"Tomas Winkler" <tomasw@...il.com>
To:	"Pierre Ossman" <drzeus-list@...eus.cx>
Cc:	"Linus Torvalds" <torvalds@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL] MMC updates for 2.6.26

On Thu, Jul 17, 2008 at 1:41 AM, Pierre Ossman <drzeus-list@...eus.cx> wrote:
> On Wed, 16 Jul 2008 15:16:26 -0700 (PDT)
> Linus Torvalds <torvalds@...ux-foundation.org> wrote:
>
>>
>>
>> On Tue, 15 Jul 2008, Pierre Ossman wrote:
>> >
>> >  45 files changed, 6054 insertions(+), 1320 deletions(-)
>>
>> If you had done the "--summary", you'd also have seen these bogus:
>>
>>        mode change 100644 => 100755 drivers/mmc/core/sdio_io.c
>>        mode change 100644 => 100755 include/linux/mmc/sdio_func.h
>>
>> entries.
>>
>> I have no idea why you think those things should be executable, but it
>> happened in commit 6d37333163025b46afbcad434ec9a5f2e88e7254 ("mmc: fix
>> sdio_io sparse errors").
>>
>
> Ouch. It was in the patch submitted to me, but I overlooked the mode
> changes there. I wasn't fully aware git could express such things in
> the patches. :)
>
>> Anyway, there's a reason why I ask people to use
>>
>>       --diffstat --summary -M
>>
>> when generating the report. This is just part of it.
>>
>> I fixed it up in the merge, so it's ok in my tree, but I thought I'd point
>> it out.
>>
>
> Thanks. I have changed my scripts and hopefully I'll notice such
> silliness in the future.
>
> Rgds
> --
>     -- Pierre Ossman

I take the blame on me. I've copied that file between OSes so it
changed executable. It slipped from my attention
Will also change my scripts as well.

Tomas
--
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/