[Change Request] fasClient fix

[Kevin Fenzi]

On Tue, 10 May 2011 14:56:56 -0700
Toshio Kuratomi <a.badger at gmail.com> wrote:

> The hotfix for fasClient that I applied just before change freeze has
> a small bug that causes SELinux labels to not be applied to the
> authorized_keys files when fasClient runs.  I'd like to apply this
> fix to the fasClient hotfix:
> 
> diff --git a/modules/hotfix/files/fas/fasClient
> b/modules/hotfix/files/fas/fasClient index 59bff7d..fe2ab6e 100755
> --- a/modules/hotfix/files/fas/fasClient
> +++ b/modules/hotfix/files/fas/fasClient
> @@ -567,7 +567,8 @@ class MakeShellAccounts(AccountSystem):
>                      os.remove(key_file)
>                  except OSError:
>                      pass
> -        subprocess.call(['/sbin/restorecon', '-R',
> os.path.join(home_dir_base, '*', '.ssh')], shell=True)
> +       subprocess.call(' '.join(['/sbin/restorecon', '-R',
> +             os.path.join(home_dir_base, '*', '.ssh')]), shell=True)
>  
>      def install_passwd_db(self):
>          '''Install the password database'''
> 
> Can I get two +1's?

Yep. +1 here. 

kevin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
Url : http://lists.fedoraproject.org/pipermail/infrastructure/attachments/20110510/60e3c17d/attachment.bin