Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/net



On Tue, May 12, 2009 at 11:03:25PM +0000, Elad Efrat wrote:
> Module Name:  src
> Committed By: elad
> Date:         Tue May 12 23:03:25 UTC 2009
> 
> Modified Files:
>       src/sys/net: if_bridge.c
> 
> Log Message:
> Move kauth(9) call before going into splnet().
> 
> Mailing list reference:
> 
>       http://mail-index.netbsd.org/tech-net/2009/05/08/msg001286.html

So,

1.  You commit without anybody having ok'd it
2.  You don't commit the patch that you sent
3.  When (publicly, even) told about an obvious bug, you still go
    ahead and commit it.

Considering your work is related to security, you should really
reconsider your methods, Elad.

(And thank Christoph for the fix.)

-- 
Quentin Garnier - cube%cubidou.net@localhost - cube%NetBSD.org@localhost
"See the look on my face from staying too long in one place
[...] every time the morning breaks I know I'm closer to falling"
KT Tunstall, Saving My Face, Drastic Fantastic, 2007.

Attachment: pgpOj32OvHqZP.pgp
Description: PGP signature



Home | Main Index | Thread Index | Old Index