[Change Request] fasClient fix

[seth vidal]

On Tue, 2011-05-10 at 14:56 -0700, Toshio Kuratomi 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?
> 

+1

-sv