add safe navigation

Signed-off-by: Corey Hemminger <hemminger@hotmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/lib/chef/resource/chef_client_config.rb b/lib/chef/resource/chef_client_config.rb
index 209380d5b2..bf5df2e998 100644
--- a/lib/chef/resource/chef_client_config.rb
+++ b/lib/chef/resource/chef_client_config.rb
@@ -252,18 +252,18 @@ class Chef
       action :create, description: "Create a client.rb config file for configuring #{ChefUtils::Dist::Infra::PRODUCT}." do
         [
           new_resource.config_directory,
-          ::File.dirname(new_resource.log_location),
+          ::File&.dirname(new_resource.log_location),
           new_resource.file_backup_path,
           new_resource.file_cache_path,
           ::File.join(new_resource.config_directory, "client.d"),
-          ::File.dirname(new_resource.pid_file)
+          ::File&.dirname(new_resource.pid_file)
         ].each do |dir_path|
           next if dir_path.nil?
           next if ::Dir.exist?(dir_path)
           directory dir_path do
             user new_resource.user unless new_resource.user.nil?
             group new_resource.group unless new_resource.group.nil?
-            mode "0750"
+            mode dir_path == ::File&.dirname(new_resource.log_location) ? "0755" : "0750"
             recursive true
           end
         end