[pve-devel] applied: [PATCH v3 cluster 2/2] cluster: use lock for legacy authkey generation

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Mar 14 09:29:50 CET 2019


On 3/13/19 3:01 PM, Fabian Gr├╝nbichler wrote:
> Signed-off-by: Fabian Gr├╝nbichler <f.gruenbichler at proxmox.com>
> ---
> Notes:
>     changes since v2:
>     - make error handling more readable
> 
>  data/PVE/Cluster.pm | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm
> index 83b401c..e641cc1 100644
> --- a/data/PVE/Cluster.pm
> +++ b/data/PVE/Cluster.pm
> @@ -158,11 +158,15 @@ sub gen_auth_key {
>  
>      check_cfs_is_mounted();
>  
> -    mkdir $authdir || $! == EEXIST || die "unable to create dir '$authdir' - $!\n";
> +    cfs_lock_authkey(undef, sub {
> +	mkdir $authdir || $! == EEXIST || die "unable to create dir '$authdir' - $!\n";
>  
> -    run_silent_cmd(['openssl', 'genrsa', '-out', $authprivkeyfn, '2048']);
> +	run_silent_cmd(['openssl', 'genrsa', '-out', $authprivkeyfn, '2048']);
>  
> -    run_silent_cmd(['openssl', 'rsa', '-in', $authprivkeyfn, '-pubout', '-out', $authpubkeyfn]);
> +	run_silent_cmd(['openssl', 'rsa', '-in', $authprivkeyfn, '-pubout', '-out', $authpubkeyfn]);
> +    });
> +
> +    die "$@\n" if $@;
>  }
>  
>  sub gen_pveca_key {
> 

applied, thanks!




More information about the pve-devel mailing list