sys-admin 1.7.1 → 1.7.6

data/Rakefile

@@ -3,7 +3,7 @@ require 'rake/clean'
 require 'rake/testtask'
 require 'rbconfig'
 
-CLEAN.include("**/*.gem", "**/*.rbx", "**/*.rbc", "ruby.core")
+CLEAN.include("**/*.gem", "**/*.rbx", "**/*.rbc", "ruby.core", "**/*.lock")
 
 namespace :gem do
   desc "Create the sys-uname gem"
@@ -11,7 +11,7 @@ namespace :gem do
     require 'rubygems/package'
     spec = eval(IO.read('sys-admin.gemspec'))
     spec.signing_key = File.join(Dir.home, '.ssh', 'gem-private_key.pem')
-    Gem::Package.build(spec, true)
+    Gem::Package.build(spec)
   end
 
   desc "Install the sys-uname gem"

data/lib/bsd/sys/admin.rb

@@ -6,10 +6,9 @@ require 'rbconfig'
 
 module Sys
   class Admin
-    private
-
     # :no-doc:
     BUF_MAX = 65536 # Max buffer for retry
+    private_constant :BUF_MAX
 
     # I'm making some aliases here to prevent potential conflicts
     attach_function :open_c, :open, [:string, :int], :int
@@ -22,8 +21,7 @@ module Sys
     attach_function :getgrnam_r, [:string, :pointer, :pointer, :size_t, :pointer], :int
     attach_function :getgrgid_r, [:long, :pointer, :pointer, :size_t, :pointer], :int
 
-    private_class_method :getlogin_r, :getpwnam_r, :getpwuid_r, :getgrnam_r
-    private_class_method :getgrgid_r
+    private_class_method :getlogin_r, :getpwnam_r, :getpwuid_r, :getgrnam_r, :getgrgid_r
     private_class_method :open_c, :pread_c, :close_c
 
     # struct passwd from /usr/include/pwd.h
@@ -48,6 +46,8 @@ module Sys
       layout(*fields)
     end
 
+    private_constant :PasswdStruct
+
     # struct group from /usr/include/grp.h
     class GroupStruct < FFI::Struct
       layout(
@@ -58,6 +58,8 @@ module Sys
       )
     end
 
+    private_constant :GroupStruct
+
     # I'm blending the timeval struct in directly here
     class LastlogStruct < FFI::Struct
       layout(
@@ -67,7 +69,7 @@ module Sys
       )
     end
 
-    public
+    private_constant :LastlogStruct
 
     # Returns the login for the current process.
     #
@@ -192,8 +194,6 @@ module Sys
       groups
     end
 
-    private
-
     # Takes a GroupStruct and converts it to a Group object.
     def self.get_group_from_struct(grp)
       Group.new do |g|
@@ -204,6 +204,8 @@ module Sys
       end
     end
 
+    private_class_method :get_group_from_struct
+
     # Takes a UserStruct and converts it to a User object.
     def self.get_user_from_struct(pwd)
       user = User.new do |u|
@@ -230,6 +232,8 @@ module Sys
       user
     end
 
+    private_class_method :get_user_from_struct
+
     # Get lastlog information for the given user.
     def self.get_lastlog_info(uid)
       logfile = '/var/log/lastlog'
@@ -252,5 +256,7 @@ module Sys
 
       lastlog
     end
+
+    private_class_method :get_lastlog_info
   end
 end

data/lib/darwin/sys/admin.rb

@@ -5,10 +5,9 @@ require 'sys/admin/common'
 
 module Sys
   class Admin
-    private
-
     # :no-doc:
     BUF_MAX = 65536 # Max buf size for retry.
+    private_constant :BUF_MAX
 
     attach_function :getlogin_r, [:pointer, :int], :int
     attach_function :getpwnam_r, [:string, :pointer, :pointer, :size_t, :pointer], :int
@@ -17,8 +16,8 @@ module Sys
     attach_function :getgrgid_r, [:long, :pointer, :pointer, :size_t, :pointer], :int
     attach_function :getlastlogx, [:long, :pointer], :pointer
 
-    private_class_method :getlogin_r, :getpwnam_r, :getpwuid_r, :getgrnam_r
-    private_class_method :getgrgid_r, :getlastlogx
+    private_class_method :getlogin_r, :getpwnam_r, :getpwuid_r
+    private_class_method :getgrnam_r, :getgrgid_r, :getlastlogx
 
     # struct passwd from /usr/include/pwd.h
     class PasswdStruct < FFI::Struct
@@ -36,6 +35,8 @@ module Sys
       )
     end
 
+    private_constant :PasswdStruct
+
     # struct group from /usr/include/grp.h
     class GroupStruct < FFI::Struct
       layout(
@@ -46,6 +47,8 @@ module Sys
       )
     end
 
+    private_constant :GroupStruct
+
     # I'm blending the timeval struct in directly here
     class LastlogxStruct < FFI::Struct
       layout(
@@ -56,7 +59,7 @@ module Sys
       )
     end
 
-    public
+    private_constant :LastlogxStruct
 
     # Returns the login for the current process.
     #
@@ -185,8 +188,6 @@ module Sys
       groups
     end
 
-    private
-
     # Takes a GroupStruct and converts it to a Group object.
     def self.get_group_from_struct(grp)
       Group.new do |g|
@@ -197,6 +198,8 @@ module Sys
       end
     end
 
+    private_class_method :get_group_from_struct
+
     # Takes a UserStruct and converts it to a User object.
     def self.get_user_from_struct(pwd)
       user = User.new do |u|
@@ -223,6 +226,8 @@ module Sys
       user
     end
 
+    private_class_method :get_user_from_struct
+
     # Gets lastlog information for the given user.
     def self.get_lastlog_info(uid)
       lastlog = LastlogxStruct.new
@@ -233,5 +238,7 @@ module Sys
 
       ptr.null? ? nil : lastlog
     end
+
+    private_class_method :get_lastlog_info
   end
 end

data/lib/linux/sys/admin.rb

@@ -5,10 +5,9 @@ require 'sys/admin/common'
 
 module Sys
   class Admin
-    private
-
     # :no-doc:
     BUF_MAX = 65536 # Absolute max buffer size for retry attempts.
+    private_constant :BUF_MAX
 
     # I'm making some aliases here to prevent potential conflicts
     attach_function :open_c, :open, [:string, :int], :int
@@ -40,6 +39,8 @@ module Sys
       )
     end
 
+    private_constant :PasswdStruct
+
     # struct group from /usr/include/grp.h
     class GroupStruct < FFI::Struct
       layout(
@@ -50,6 +51,8 @@ module Sys
       )
     end
 
+    private_constant :GroupStruct
+
     # I'm blending the timeval struct in directly here
     class LastlogStruct < FFI::Struct
       layout(
@@ -59,18 +62,12 @@ module Sys
       )
     end
 
-    public
+    private_constant :LastlogStruct
 
     # Returns the login for the current process.
     #
     def self.get_login
-      buf = FFI::MemoryPointer.new(:char, 256)
-
-      if getlogin_r(buf, buf.size) != 0
-        raise Error, "getlogin_r function failed: " + strerror(FFI.errno)
-      end
-
-      buf.read_string
+      get_user(geteuid()).name
     end
 
     # Returns a User object for the given name or uid. Raises an error
@@ -204,8 +201,6 @@ module Sys
       groups
     end
 
-    private
-
     # Takes a GroupStruct and converts it to a Group object.
     def self.get_group_from_struct(grp)
       Group.new do |g|
@@ -216,6 +211,8 @@ module Sys
       end
     end
 
+    private_class_method :get_group_from_struct
+
     # Takes a UserStruct and converts it to a User object.
     def self.get_user_from_struct(pwd)
       user = User.new do |u|
@@ -242,6 +239,8 @@ module Sys
       user
     end
 
+    private_class_method :get_user_from_struct
+
     # Note: it seems that Linux, or at least Ubuntu, does not track logins
     # via GDM (Gnome Display Manager) for some reason, so this may not return
     # anything useful.
@@ -269,5 +268,7 @@ module Sys
 
       lastlog
     end
+
+    private_class_method :get_lastlog_info
   end
 end

data/lib/sunos/sys/admin.rb

@@ -5,10 +5,9 @@ require 'sys/admin/common'
 
 module Sys
   class Admin
-    private
-
     # :no-doc:
     BUF_MAX = 65536 # Max buffer size for retry.
+    private_constant :BUF_MAX
 
     # I'm making some aliases here to prevent potential conflicts
     attach_function :open_c, :open, [:string, :int], :int
@@ -42,6 +41,8 @@ module Sys
       )
     end
 
+    private_constant :PasswdStruct
+
     # struct group from /usr/include/grp.h
     class GroupStruct < FFI::Struct
       layout(
@@ -52,6 +53,8 @@ module Sys
       )
     end
 
+    private_constant :GroupStruct
+
     # I'm blending the timeval struct in directly here
     class LastlogStruct < FFI::Struct
       layout(
@@ -61,7 +64,7 @@ module Sys
       )
     end
 
-    public
+    private_constant :LastlogStruct
 
     # Returns the login for the current process.
     #
@@ -194,8 +197,6 @@ module Sys
       groups
     end
 
-    private
-
     # Takes a GroupStruct and converts it to a Group object.
     def self.get_group_from_struct(grp)
       Group.new do |g|
@@ -206,6 +207,8 @@ module Sys
       end
     end
 
+    private_class_method :get_group_from_struct
+
     # Takes a UserStruct and converts it to a User object.
     def self.get_user_from_struct(pwd)
       user = User.new do |u|
@@ -232,6 +235,8 @@ module Sys
       user
     end
 
+    private_class_method :get_use_from_struct
+
     # The use of pread was necessary here because it's a sparse file. Note
     # also that while Solaris supports the getuserattr function, it doesn't
     # appear to store anything regarding login information.
@@ -257,5 +262,7 @@ module Sys
 
       lastlog
     end
+
+    private_class_method :get_lastlog_info
   end
 end

data/lib/sys/admin.rb

@@ -1,7 +1,7 @@
 module Sys
   class Admin
     # The version of the sys-admin library.
-    VERSION = '1.7.1'.freeze
+    VERSION = '1.7.6'.freeze
   end
 end
 

data/lib/sys/admin/common.rb

@@ -9,12 +9,11 @@ module Sys
     extend FFI::Library
     ffi_lib FFI::Library::LIBC
 
-    private
-
     attach_function :strerror, [:int], :string
 
     attach_function :getlogin, [], :string
     attach_function :getuid, [], :long
+    attach_function :geteuid, [], :long
     attach_function :getpwnam, [:string], :pointer
     attach_function :getpwuid, [:long], :pointer
     attach_function :getpwent, [], :pointer
@@ -27,12 +26,10 @@ module Sys
     attach_function :endgrent, [], :void
     attach_function :setgrent, [], :void
 
-    private_class_method :getlogin, :getuid, :getpwnam, :getpwuid, :getpwent
-    private_class_method :setpwent, :endpwent, :getgrgid, :getgrnam
+    private_class_method :getlogin, :getuid, :geteuid, :getpwnam, :getpwuid
+    private_class_method :getpwent, :setpwent, :endpwent, :getgrgid, :getgrnam
     private_class_method :getgrent, :endgrent, :setgrent, :strerror
 
-    public
-
     # Error typically raised if any of the Sys::Admin methods fail.
     class Error < StandardError; end
 

data/lib/unix/sys/admin.rb

@@ -5,8 +5,6 @@ require 'sys/admin/common'
 
 module Sys
   class Admin
-    private
-
     class PasswdStruct < FFI::Struct
       layout(
         :pw_name,   :string,
@@ -19,6 +17,8 @@ module Sys
       )
     end
 
+    private_constant :PasswdStruct
+
     class GroupStruct < FFI::Struct
       layout(
         :gr_name,   :string,
@@ -28,7 +28,7 @@ module Sys
       )
     end
 
-    public
+    private_constant :GroupStruct
 
     # Returns the login for the current process.
     #
@@ -133,8 +133,6 @@ module Sys
       groups
     end
 
-    private
-
     # Takes a GroupStruct and converts it to a Group object.
     def self.get_group_from_struct(grp)
       Group.new do |g|
@@ -145,6 +143,8 @@ module Sys
       end
     end
 
+    private_class_method :get_group_from_struct
+
     # Takes a UserStruct and converts it to a User object.
     def self.get_user_from_struct(pwd)
       user = User.new do |u|
@@ -159,5 +159,7 @@ module Sys
 
       user
     end
+
+    private_class_method :get_user_from_struct
   end
 end

data/lib/windows/sys/admin.rb

@@ -97,8 +97,6 @@ module Sys
               @sid_type = "unknown"
         end
       end
-
-      @sid_type
     end
   end
 
@@ -314,9 +312,8 @@ module Sys
     SidTypeUnknown        = 8
     SidTypeComputer       = 9
 
-    private
-
     HKEY = "SOFTWARE\\Microsoft\\Windows NT\\CurrentVersion\\ProfileList\\"
+    private_constant :HKEY
 
     # Retrieves the user's home directory. For local accounts query the
     # registry. For domain accounts use ADSI and use the HomeDirectory.
@@ -343,6 +340,8 @@ module Sys
       dir
     end
 
+    private_class_method :get_home_dir
+
     # A private method that lower cases all keys, and converts them
     # all to symbols.
     #
@@ -357,6 +356,8 @@ module Sys
       rhash
     end
 
+    private_class_method :munge_options
+
     # An internal, private method for getting a list of groups for
     # a particular user. The first member is a list of group names,
     # the second member is the primary group ID.
@@ -368,6 +369,8 @@ module Sys
       [array, adsi.PrimaryGroupId]
     end
 
+    private_class_method :get_groups
+
     # An internal, private method for getting a list of members for
     # any particular group.
     #
@@ -378,13 +381,13 @@ module Sys
       array
     end
 
+    private_class_method :get_members
+
     # Used by the get_login method
     ffi_lib :advapi32
     attach_function :GetUserNameW, [:pointer, :pointer], :bool
     private_class_method :GetUserNameW
 
-    public
-
     # Creates the given +user+. If no domain option is specified,
     # then it defaults to your local host, i.e. a local account is
     # created.
@@ -705,7 +708,7 @@ module Sys
         end
       }
 
-      if usr.kind_of?(Fixnum)
+      if usr.kind_of?(Integer)
         query << " and sid like '%-#{usr}'"
       else
         query << " and name = '#{usr}'"
@@ -718,7 +721,7 @@ module Sys
 
         # Because our 'like' query isn't fulproof, let's parse
         # the SID again to make sure
-        if usr.kind_of?(Fixnum)
+        if usr.kind_of?(Integer)
           next if usr != uid
         end
 
@@ -886,7 +889,7 @@ module Sys
         end
       }
 
-      if grp.kind_of?(Fixnum)
+      if grp.kind_of?(Integer)
         query << " and sid like '%-#{grp}'"
       else
         query << " and name = '#{grp}'"
@@ -899,7 +902,7 @@ module Sys
 
         # Because our 'like' query isn't fulproof, let's parse
         # the SID again to make sure
-        if grp.kind_of?(Fixnum)
+        if grp.kind_of?(Integer)
           next if grp != gid
         end