Patch matchpathcon to eliminate file "/"

Patch matchpathcon to eliminate file "/"

[Daniel J Walsh]

[-- Attachment #1: Type: text/plain, Size: 369 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

So

matchpathcon /etc/
Will work the same as
matchpathcon /etc
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAkmu5wMACgkQrlYvE4MpobNK4gCgiVeXXEZcCMsJKXM7jqh6r1u3
OScAoLcmXBIR63gpvA8RS3g07pcPC6IF
=e+Re
-----END PGP SIGNATURE-----

[-- Attachment #2: libselinux-matchpathcon.patch --]
[-- Type: text/plain, Size: 422 bytes --]

--- nsalibselinux/utils/matchpathcon.c	2008-10-28 10:06:51.000000000 -0400
+++ libselinux-2.0.78/utils/matchpathcon.c	2009-03-04 15:23:52.000000000 -0500
@@ -101,6 +101,11 @@
 	for (i = optind; i < argc; i++) {
 		int mode = 0;
 		struct stat buf;
+		int len = strlen(argv[i]);
+		if (len > 1  && argv[i][len - 1 ] == '/') {
+			argv[i][len - 1 ] = '\0';
+		}
+
 		if (lstat(argv[i], &buf) == 0)
 			mode = buf.st_mode;
 

[-- Attachment #3: libselinux-matchpathcon.patch.sig --]
[-- Type: application/pgp-signature, Size: 72 bytes --]

Re: Patch matchpathcon to eliminate file "/"

[Chad Sellers]

On 3/4/09 3:39 PM, "Daniel J Walsh" <dwalsh@redhat.com> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> So
> 
> matchpathcon /etc/
> Will work the same as
> matchpathcon /etc
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org
> 
> iEYEARECAAYFAkmu5wMACgkQrlYvE4MpobNK4gCgiVeXXEZcCMsJKXM7jqh6r1u3
> OScAoLcmXBIR63gpvA8RS3g07pcPC6IF
> =e+Re
> -----END PGP SIGNATURE-----
> --- nsalibselinux/utils/matchpathcon.c 2008-10-28 10:06:51.000000000 -0400
> +++ libselinux-2.0.78/utils/matchpathcon.c 2009-03-04 15:23:52.000000000 -0500
> @@ -101,6 +101,11 @@
> for (i = optind; i < argc; i++) {
> int mode = 0;
> struct stat buf;
> +  int len = strlen(argv[i]);
> +  if (len > 1  && argv[i][len - 1 ] == '/') {
> +   argv[i][len - 1 ] = '\0';
> +  }
> +
> if (lstat(argv[i], &buf) == 0)
> mode = buf.st_mode;
>  

Looks fine to me.

Acked-by: Chad Sellers <csellers@tresys.com>



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

Re: Patch matchpathcon to eliminate file "/"

[Joshua Brindle]

Chad Sellers wrote:
> On 3/4/09 3:39 PM, "Daniel J Walsh" <dwalsh@redhat.com> wrote:
> 
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> So
>>
>> matchpathcon /etc/
>> Will work the same as
>> matchpathcon /etc
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.9 (GNU/Linux)
>> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org
>>
>> iEYEARECAAYFAkmu5wMACgkQrlYvE4MpobNK4gCgiVeXXEZcCMsJKXM7jqh6r1u3
>> OScAoLcmXBIR63gpvA8RS3g07pcPC6IF
>> =e+Re
>> -----END PGP SIGNATURE-----
>> --- nsalibselinux/utils/matchpathcon.c 2008-10-28 10:06:51.000000000 -0400
>> +++ libselinux-2.0.78/utils/matchpathcon.c 2009-03-04 15:23:52.000000000 -0500
>> @@ -101,6 +101,11 @@
>> for (i = optind; i < argc; i++) {
>> int mode = 0;
>> struct stat buf;
>> +  int len = strlen(argv[i]);
>> +  if (len > 1  && argv[i][len - 1 ] == '/') {
>> +   argv[i][len - 1 ] = '\0';
>> +  }
>> +
>> if (lstat(argv[i], &buf) == 0)
>> mode = buf.st_mode;
>>  
> 
> Looks fine to me.
> 
> Acked-by: Chad Sellers <csellers@tresys.com>
> 

Merged in libselinux-2.0.81

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.