Re: [Change Request] fasClient fix

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

 



On Tue, 10 May 2011 14:56:56 -0700
Toshio Kuratomi <a.badger@xxxxxxxxx> 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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux