linux-kernel - Re: [PATCH 8/8] One Laptop Per Child power/battery driver

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: <1179034789.2802.61.camel@shinybook.infradead.org>
Date:	Sun, 13 May 2007 13:39:48 +0800
From:	David Woodhouse <dwmw2@...radead.org>
To:	Pete Zaitcev <zaitcev@...hat.com>
Cc:	"Maciej W. Rozycki" <macro-6z/3iImG2C8G8FEW9MqTrA@...lic.gmane.org>,
	Andrew Morton <akpm-3NddpPZAyC0@...lic.gmane.org>,
	kernel-discuss-CN5wO63fgwogsBAKwltoeQ@...lic.gmane.org,
	Greg KH <gregkh-l3A5Bk7waGM@...lic.gmane.org>,
	linux-kernel@...r.kernel.org,
	Anton Vorontsov <cbou-JGs/UdohzUI@...lic.gmane.org>,
	Pavel Machek <pavel-+ZI9xUNit7I@...lic.gmane.org>,
	Satyam Sharma 
	<satyam.sharma-Re5JQEeQqe8AvxtiuMwx3w@...lic.gmane.org>
Subject: Re: [PATCH 8/8] One Laptop Per Child power/battery driver

Please don't use my work email address unless I _really_ need to be
wearing my tinfoil hat when I read it.

On Sat, 2007-05-12 at 09:35 -0700, Pete Zaitcev wrote:
> On Tue, 08 May 2007 18:52:31 +0100, David Woodhouse <dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@...lic.gmane.org> wrote:
> > On Tue, 2007-05-08 at 18:45 +0100, Maciej W. Rozycki wrote:
> 
> > >  Well, my name has an acute above "o" and a dot above "z", but I do not 
> > > have such a high expectation as to have it correctly spelled in comments 
> > > and elsewhere within some code before I am able to get it right in 
> > > official documents issued in English.  And the universe is not going to 
> > > collapse because of the missing accents, so I consider this an issue we 
> > > can think of resolving once we have fixed all the bugs that we have.
> > 
> > It's not an issue which needs resolving. We can spell your name just
> > fine; we just need Pavel to quit whining and join us in the 21st
> > century.
> 
> I hate to agree with Pavel, but the real issue here is you being
> intentionally obtuse. Naked patches are not compatible with UTF-8.

Wrong. Of course they're compatible -- they're just applied as a byte
stream. Patch and git are agnostic about file _contents_.

Git's apply-mailbox tool does honour the Content-Type: header in the
mail, and it converts _comments_ to the correct charset for the
repository, as well as handling RFC2047-encoding in the From: header.
But it doesn't touch the patch itself.

> To assume that UTF-8 is not converted anywhere is tantamount to the
> fallacy of SPF: Everyone on the net must do foo, then nirvana happens
> If the code is being pulled git to git, UTF-8 is fine. Or send your
> patches in MIME at the very least.

Some systems mangle whitespace in transit too, or even convert to HTML.
I've known people who _have_ to send patches as MIME attachments to
avoid having their patches converted to HTML. By your logic, we could
argue that we shouldn't send patches in email at all, because it's also
"tantamount to the fallacy of SPF".

But we'd be wrong -- just as you are. SPF is utterly moronic because it
suddenly tries to change the rules and assumes that everyone has already
changed, or it throws away valid email from unknown senders. In the case
of sending patches, yes you _do_ assume that people aren't doing bizarre
things to mangle them in transit, but that mangling is actually
relatively rare. Not only that, but you only really care about a
_single_ transit path, from you to whoever applies the patch. Which is
easy enough to debug if it starts to do stupid things like converting to
HTML or converting charsets.

-- 
dwmw2

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