Re: [PATCH] (improved) LSM root_plug fixup

gregat_private

On Tue, Aug 05, 2003 at 11:58:40AM +0200, bert hubert wrote:
> Stephen Smalley noticed a typo in my new Makefile and suggested rediffing
> against bitkeeper which now has SELinux merged. 
> 
> Further changes, added small comment, added module license & description,
> actually tested compiling the kernel with different security settings.
> 
> Please consider applying.

Sorry for the long delay, I've been busy with other stuff.

Yes, you are correct, this kind of change is necessary in order to get
root_plug working again.

I only have one very tiny, cosmetic, problem with the patch:

> --- linux-2.6.0-test2-bk/security/capability.c	2003-08-05 09:45:07.000000000 +0200
> +++ linux-2.6.0-test2-bk-ahu/security/capability.c	2003-08-05 11:03:58.000000000 +0200
> @@ -6,6 +6,7 @@
>   *	the Free Software Foundation; either version 2 of the License, or
>   *	(at your option) any later version.
>   *
> + *	2003-08-05	Split out common functions to commoncap.c (ahuat_private)
>   */
>  
>  #include <linux/config.h>
> @@ -23,333 +24,6 @@

and:

> --- linux-2.6.0-test2-bk/security/commoncap.c	1970-01-01 01:00:00.000000000 +0100
> +++ linux-2.6.0-test2-bk-ahu/security/commoncap.c	2003-08-05 11:04:01.000000000 +0200
> @@ -0,0 +1,354 @@
> +/* Common capabilities, needed by capability.o and root_cap.o 
> + *
> + *	This program is free software; you can redistribute it and/or modify
> + *	it under the terms of the GNU General Public License as published by
> + *	the Free Software Foundation; either version 2 of the License, or
> + *	(at your option) any later version.
> + *
> + *	2003-08-05	Split out from capability.c (ahuat_private)
> + */

Please do not add "changelog" type comments to files that do not already
have changelogs in them.  They are unmaintainable within the kernel and
do nothing to help anyone out.

Rely on the change history that we now have due to the use of bitkeeper,
which shows your name associated with a specific changeset.

If you take these lines out, I have no problem with the patch.

thanks,

greg k-h
_______________________________________________
linux-security-module mailing list
linux-security-moduleat_private
http://mail.wirex.com/mailman/listinfo/linux-security-module