diff --git a/modules/net_share/lib/puppet/provider/net_share/net_share.rb b/modules/net_share/lib/puppet/provider/net_share/net_share.rb
index 630e238..1fcc485 100644
--- a/modules/net_share/lib/puppet/provider/net_share/net_share.rb
+++ b/modules/net_share/lib/puppet/provider/net_share/net_share.rb
@@ -73,7 +73,7 @@ def self.query(name)
 
     properties[:ensure] = :present
 
-    output.each do |line|
+    output.each_line do |line|
       break if line.rstrip.length == 0
 
       last_name = name
diff --git a/modules/net_share/lib/puppet/type/net_share.rb b/modules/net_share/lib/puppet/type/net_share.rb
index ce4171e..1a6e5f5 100644
--- a/modules/net_share/lib/puppet/type/net_share.rb
+++ b/modules/net_share/lib/puppet/type/net_share.rb
@@ -51,10 +51,8 @@ def property_matches?(current, desired)
     desc "An array of permissions. Example: ['computer\\user,full', 'computer\\user2,change', 'computer\\user3,read']"
 
     munge do |value|
-      value.collect do |item|
-        user, access = item.split(',', 2)
-        "#{user.strip},#{access.strip.downcase}"
-      end
+      user, access = value.split(',', 2)
+      "#{user.strip},#{access.strip.downcase}"
     end
   end