linux-kernel - Re: [PULL] v9fs bug fixes and documentation 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]

Message-ID: <alpine.LFD.1.10.0805141930530.3019@woody.linux-foundation.org>
Date:	Wed, 14 May 2008 19:40:02 -0700 (PDT)
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	Eric Van Hensbergen <ericvh@...il.com>
cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	V9FS Developers <v9fs-developer@...ts.sourceforge.net>
Subject: Re: [PULL] v9fs bug fixes and documentation updates for 2.6.26

On Wed, 14 May 2008, Eric Van Hensbergen wrote:
> 
> Jim Meyering (1):
>       fs/9p/v9fs.c (v9fs_parse_options): Handle kstrdup and
> match_strdup failure. Now that this function can fail, return an int,
> diagnose other option-parsing failures, and adjust the sole caller:
> (v9fs_session_init): Handle kstrdup failure. Propagate any new
> v9fs_parse_options failure "up".

Grr. Somebody isn't following the nice rules we have and that git 
encourages: make a commit message be a nice "one-line header" with the 
more complete explanation separated by an empty line (and with nice 
line-breaks etc).

IOW, womebody has bad habits from SCM systems that didn't have good 
visualization tools, so likely nobody actually looked at the messages 
anyway.

In fact, looking closer, I don't even know how you did that. The thing 
must have been forwarded as email, because the author is Jim Meyering, but 
it's been forwarded through Andrew etc, so I bet the message was fine at 
some point. And all the git email-application tools do the rigth thing. 

Anyway, I don't *quite* care enough to ask you to re-do this, and I merged 
it, but I do actually enjoy the fact that our commit messages are 
generally pretty readable, and the one-liner header rule means that gitk 
and various shortlog tools all give nice summaries too.

So please look out for this in the future.

There are other commits that don't follow the rules, btw: you've applied 
patches from others that have the rigth Sign-off's from the authors, but 
you should sign off yourself, not just ack them. By actually taking that 
thing and committing it, you're doing more than ack'ing somebody elses 
work.

I know it isn't really a big deal, especially with those things mostly 
being pretty trivial, but I'd rather nip these issues in the bud than have 
it come back when it actually might matter.

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