chef 0.10.10 → 10.12.0.rc.1

data/lib/chef/win32/api/memory.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module Memory
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Constants
@@ -56,21 +56,21 @@ HLOCAL WINAPI LocalAlloc(
   __in  SIZE_T uBytes
 );
 =end
-        attach_function :LocalAlloc, [ :UINT, :SIZE_T ], :pointer
+        safe_attach_function :LocalAlloc, [ :UINT, :SIZE_T ], :pointer
 
 =begin
 UINT WINAPI LocalFlags(
   __in  HLOCAL hMem
 );
 =end
-        attach_function :LocalFlags, [ :pointer ], :UINT
+        safe_attach_function :LocalFlags, [ :pointer ], :UINT
 
 =begin
 HLOCAL WINAPI LocalFree(
   __in  HLOCAL hMem
 );
 =end
-        attach_function :LocalFree, [ :pointer ], :pointer
+        safe_attach_function :LocalFree, [ :pointer ], :pointer
 
 =begin
 HLOCAL WINAPI LocalReAlloc(
@@ -79,25 +79,25 @@ HLOCAL WINAPI LocalReAlloc(
   __in  UINT uFlags
 );
 =end
-        attach_function :LocalReAlloc, [ :pointer, :SIZE_T, :UINT ], :pointer
+        safe_attach_function :LocalReAlloc, [ :pointer, :SIZE_T, :UINT ], :pointer
 
 =begin
 UINT WINAPI LocalSize(
   __in  HLOCAL hMem
 );
 =end
-        attach_function :LocalSize, [ :pointer ], :SIZE_T
+        safe_attach_function :LocalSize, [ :pointer ], :SIZE_T
 
         ###############################################
         # FFI API Bindings
         ###############################################
 
         ffi_lib FFI::Library::LIBC
-        attach_function :malloc, [:size_t], :pointer
-        attach_function :calloc, [:size_t], :pointer
-        attach_function :realloc, [:pointer, :size_t], :pointer
-        attach_function :free, [:pointer], :void
-        attach_function :memcpy, [:pointer, :pointer, :size_t], :pointer
+        safe_attach_function :malloc, [:size_t], :pointer
+        safe_attach_function :calloc, [:size_t], :pointer
+        safe_attach_function :realloc, [:pointer, :size_t], :pointer
+        safe_attach_function :free, [:pointer], :void
+        safe_attach_function :memcpy, [:pointer, :pointer, :size_t], :pointer
 
       end
     end

data/lib/chef/win32/api/process.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module Process
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Bindings
@@ -30,9 +30,9 @@ class Chef
 
         ffi_lib 'kernel32'
 
-        attach_function :GetCurrentProcess, [], :HANDLE
-        attach_function :GetProcessHandleCount, [ :HANDLE, :LPDWORD ], :BOOL
-        attach_function :GetProcessId, [ :HANDLE ], :DWORD
+        safe_attach_function :GetCurrentProcess, [], :HANDLE
+        safe_attach_function :GetProcessHandleCount, [ :HANDLE, :LPDWORD ], :BOOL
+        safe_attach_function :GetProcessId, [ :HANDLE ], :DWORD
 
       end
     end

data/lib/chef/win32/api/psapi.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module PSAPI
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Bindings
@@ -43,7 +43,7 @@ class Chef
 
         ffi_lib 'psapi'
 
-        attach_function :GetProcessMemoryInfo, [ :HANDLE, :pointer, :DWORD ], :BOOL
+        safe_attach_function :GetProcessMemoryInfo, [ :HANDLE, :pointer, :DWORD ], :BOOL
 
       end
     end

data/lib/chef/win32/api/security.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module Security
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Constants
@@ -295,45 +295,45 @@ class Chef
 
         ffi_lib "advapi32"
 
-        attach_function :AddAce, [ :pointer, :DWORD, :DWORD, :LPVOID, :DWORD ], :BOOL
-        attach_function :AddAccessAllowedAce, [ :pointer, :DWORD, :DWORD, :pointer ], :BOOL
-        attach_function :AddAccessAllowedAceEx, [ :pointer, :DWORD, :DWORD, :DWORD, :pointer ], :BOOL
-        attach_function :AddAccessDeniedAce, [ :pointer, :DWORD, :DWORD, :pointer ], :BOOL
-        attach_function :AddAccessDeniedAceEx, [ :pointer, :DWORD, :DWORD, :DWORD, :pointer ], :BOOL
-        attach_function :AdjustTokenPrivileges, [ :HANDLE, :BOOL, :pointer, :DWORD, :pointer, :PDWORD ], :BOOL
-        attach_function :ConvertSidToStringSidA, [ :pointer, :pointer ], :BOOL
-        attach_function :ConvertStringSidToSidW, [ :pointer, :pointer ], :BOOL
-        attach_function :DeleteAce, [ :pointer, :DWORD ], :BOOL
-        attach_function :EqualSid, [ :pointer, :pointer ], :BOOL
-        attach_function :FreeSid, [ :pointer ], :pointer
-        attach_function :GetAce, [ :pointer, :DWORD, :pointer ], :BOOL
-        attach_function :GetLengthSid, [ :pointer ], :DWORD
-        attach_function :GetNamedSecurityInfoW,  [ :LPWSTR, :SE_OBJECT_TYPE, :DWORD, :pointer, :pointer, :pointer, :pointer, :pointer ], :DWORD
-        attach_function :GetSecurityDescriptorControl, [ :pointer, :PWORD, :LPDWORD], :BOOL
-        attach_function :GetSecurityDescriptorDacl, [ :pointer, :LPBOOL, :pointer, :LPBOOL ], :BOOL
-        attach_function :GetSecurityDescriptorGroup, [ :pointer, :pointer, :LPBOOL], :BOOL
-        attach_function :GetSecurityDescriptorOwner, [ :pointer, :pointer, :LPBOOL], :BOOL
-        attach_function :GetSecurityDescriptorSacl, [ :pointer, :LPBOOL, :pointer, :LPBOOL ], :BOOL
-        attach_function :InitializeAcl, [ :pointer, :DWORD, :DWORD ], :BOOL
-        attach_function :InitializeSecurityDescriptor, [ :pointer, :DWORD ], :BOOL
-        attach_function :IsValidAcl, [ :pointer ], :BOOL
-        attach_function :IsValidSecurityDescriptor, [ :pointer ], :BOOL
-        attach_function :IsValidSid, [ :pointer ], :BOOL
-        attach_function :LookupAccountNameW, [ :LPCWSTR, :LPCWSTR, :pointer, :LPDWORD, :LPWSTR, :LPDWORD, :pointer ], :BOOL
-        attach_function :LookupAccountSidW, [ :LPCWSTR, :pointer, :LPWSTR, :LPDWORD, :LPWSTR, :LPDWORD, :pointer ], :BOOL
-        attach_function :LookupPrivilegeNameW, [ :LPCWSTR, :PLUID, :LPWSTR, :LPDWORD ], :BOOL
-        attach_function :LookupPrivilegeDisplayNameW, [ :LPCWSTR, :LPCWSTR, :LPWSTR, :LPDWORD, :LPDWORD ], :BOOL
-        attach_function :LookupPrivilegeValueW, [ :LPCWSTR, :LPCWSTR, :PLUID ], :BOOL
-        attach_function :MakeAbsoluteSD, [ :pointer, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD], :BOOL
-        attach_function :OpenProcessToken, [ :HANDLE, :DWORD, :PHANDLE ], :BOOL
-        attach_function :QuerySecurityAccessMask, [ :DWORD, :LPDWORD ], :void
-        attach_function :SetFileSecurityW, [ :LPWSTR, :DWORD, :pointer ], :BOOL
-        attach_function :SetNamedSecurityInfoW, [ :LPWSTR, :SE_OBJECT_TYPE, :DWORD, :pointer, :pointer, :pointer, :pointer ], :DWORD
-        attach_function :SetSecurityAccessMask, [ :DWORD, :LPDWORD ], :void
-        attach_function :SetSecurityDescriptorDacl, [ :pointer, :BOOL, :pointer, :BOOL ], :BOOL
-        attach_function :SetSecurityDescriptorGroup, [ :pointer, :pointer, :BOOL ], :BOOL
-        attach_function :SetSecurityDescriptorOwner, [ :pointer, :pointer, :BOOL ], :BOOL
-        attach_function :SetSecurityDescriptorSacl, [ :pointer, :BOOL, :pointer, :BOOL ], :BOOL
+        safe_attach_function :AddAce, [ :pointer, :DWORD, :DWORD, :LPVOID, :DWORD ], :BOOL
+        safe_attach_function :AddAccessAllowedAce, [ :pointer, :DWORD, :DWORD, :pointer ], :BOOL
+        safe_attach_function :AddAccessAllowedAceEx, [ :pointer, :DWORD, :DWORD, :DWORD, :pointer ], :BOOL
+        safe_attach_function :AddAccessDeniedAce, [ :pointer, :DWORD, :DWORD, :pointer ], :BOOL
+        safe_attach_function :AddAccessDeniedAceEx, [ :pointer, :DWORD, :DWORD, :DWORD, :pointer ], :BOOL
+        safe_attach_function :AdjustTokenPrivileges, [ :HANDLE, :BOOL, :pointer, :DWORD, :pointer, :PDWORD ], :BOOL
+        safe_attach_function :ConvertSidToStringSidA, [ :pointer, :pointer ], :BOOL
+        safe_attach_function :ConvertStringSidToSidW, [ :pointer, :pointer ], :BOOL
+        safe_attach_function :DeleteAce, [ :pointer, :DWORD ], :BOOL
+        safe_attach_function :EqualSid, [ :pointer, :pointer ], :BOOL
+        safe_attach_function :FreeSid, [ :pointer ], :pointer
+        safe_attach_function :GetAce, [ :pointer, :DWORD, :pointer ], :BOOL
+        safe_attach_function :GetLengthSid, [ :pointer ], :DWORD
+        safe_attach_function :GetNamedSecurityInfoW,  [ :LPWSTR, :SE_OBJECT_TYPE, :DWORD, :pointer, :pointer, :pointer, :pointer, :pointer ], :DWORD
+        safe_attach_function :GetSecurityDescriptorControl, [ :pointer, :PWORD, :LPDWORD], :BOOL
+        safe_attach_function :GetSecurityDescriptorDacl, [ :pointer, :LPBOOL, :pointer, :LPBOOL ], :BOOL
+        safe_attach_function :GetSecurityDescriptorGroup, [ :pointer, :pointer, :LPBOOL], :BOOL
+        safe_attach_function :GetSecurityDescriptorOwner, [ :pointer, :pointer, :LPBOOL], :BOOL
+        safe_attach_function :GetSecurityDescriptorSacl, [ :pointer, :LPBOOL, :pointer, :LPBOOL ], :BOOL
+        safe_attach_function :InitializeAcl, [ :pointer, :DWORD, :DWORD ], :BOOL
+        safe_attach_function :InitializeSecurityDescriptor, [ :pointer, :DWORD ], :BOOL
+        safe_attach_function :IsValidAcl, [ :pointer ], :BOOL
+        safe_attach_function :IsValidSecurityDescriptor, [ :pointer ], :BOOL
+        safe_attach_function :IsValidSid, [ :pointer ], :BOOL
+        safe_attach_function :LookupAccountNameW, [ :LPCWSTR, :LPCWSTR, :pointer, :LPDWORD, :LPWSTR, :LPDWORD, :pointer ], :BOOL
+        safe_attach_function :LookupAccountSidW, [ :LPCWSTR, :pointer, :LPWSTR, :LPDWORD, :LPWSTR, :LPDWORD, :pointer ], :BOOL
+        safe_attach_function :LookupPrivilegeNameW, [ :LPCWSTR, :PLUID, :LPWSTR, :LPDWORD ], :BOOL
+        safe_attach_function :LookupPrivilegeDisplayNameW, [ :LPCWSTR, :LPCWSTR, :LPWSTR, :LPDWORD, :LPDWORD ], :BOOL
+        safe_attach_function :LookupPrivilegeValueW, [ :LPCWSTR, :LPCWSTR, :PLUID ], :BOOL
+        safe_attach_function :MakeAbsoluteSD, [ :pointer, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD, :pointer, :LPDWORD], :BOOL
+        safe_attach_function :OpenProcessToken, [ :HANDLE, :DWORD, :PHANDLE ], :BOOL
+        safe_attach_function :QuerySecurityAccessMask, [ :DWORD, :LPDWORD ], :void
+        safe_attach_function :SetFileSecurityW, [ :LPWSTR, :DWORD, :pointer ], :BOOL
+        safe_attach_function :SetNamedSecurityInfoW, [ :LPWSTR, :SE_OBJECT_TYPE, :DWORD, :pointer, :pointer, :pointer, :pointer ], :DWORD
+        safe_attach_function :SetSecurityAccessMask, [ :DWORD, :LPDWORD ], :void
+        safe_attach_function :SetSecurityDescriptorDacl, [ :pointer, :BOOL, :pointer, :BOOL ], :BOOL
+        safe_attach_function :SetSecurityDescriptorGroup, [ :pointer, :pointer, :BOOL ], :BOOL
+        safe_attach_function :SetSecurityDescriptorOwner, [ :pointer, :pointer, :BOOL ], :BOOL
+        safe_attach_function :SetSecurityDescriptorSacl, [ :pointer, :BOOL, :pointer, :BOOL ], :BOOL
 
       end
     end

data/lib/chef/win32/api/system.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module System
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Constants
@@ -153,20 +153,20 @@ BOOL WINAPI CloseHandle(
   __in  HANDLE hObject
 );
 =end
-        attach_function :CloseHandle, [ :HANDLE ], :BOOL
+        safe_attach_function :CloseHandle, [ :HANDLE ], :BOOL
 
 =begin
 DWORD WINAPI GetVersion(void);
 =end
-        attach_function :GetVersion, [], :DWORD
+        safe_attach_function :GetVersion, [], :DWORD
 
 =begin
 BOOL WINAPI GetVersionEx(
   __inout  LPOSVERSIONINFO lpVersionInfo
 );
 =end
-        attach_function :GetVersionExW, [:pointer], :BOOL
-        attach_function :GetVersionExA, [:pointer], :BOOL
+        safe_attach_function :GetVersionExW, [:pointer], :BOOL
+        safe_attach_function :GetVersionExA, [:pointer], :BOOL
 
 =begin
 BOOL WINAPI GetProductInfo(
@@ -177,14 +177,14 @@ BOOL WINAPI GetProductInfo(
   __out  PDWORD pdwReturnedProductType
 );
 =end
-        attach_function :GetProductInfo, [:DWORD, :DWORD, :DWORD, :DWORD, :PDWORD], :BOOL
+        safe_attach_function :GetProductInfo, [:DWORD, :DWORD, :DWORD, :DWORD, :PDWORD], :BOOL
 
 =begin
 int WINAPI GetSystemMetrics(
   __in  int nIndex
 );
 =end
-        attach_function :GetSystemMetrics, [:int], :int
+        safe_attach_function :GetSystemMetrics, [:int], :int
 
       end
     end

data/lib/chef/win32/api/unicode.rb

@@ -19,10 +19,10 @@
 require 'chef/win32/api'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     module API
       module Unicode
-        extend Chef::Win32::API
+        extend Chef::ReservedNames::Win32::API
 
         ###############################################
         # Win32 API Constants
@@ -101,7 +101,7 @@ BOOL IsTextUnicode(
   __inout  LPINT lpiResult
 );
 =end
-        attach_function :IsTextUnicode, [:pointer, :int, :LPINT], :BOOL
+        safe_attach_function :IsTextUnicode, [:pointer, :int, :LPINT], :BOOL
 
 =begin
 int MultiByteToWideChar(
@@ -113,7 +113,7 @@ int MultiByteToWideChar(
   __in   int cchWideChar
 );
 =end
-        attach_function :MultiByteToWideChar, [:UINT, :DWORD, :LPCSTR, :int, :LPWSTR, :int], :int
+        safe_attach_function :MultiByteToWideChar, [:UINT, :DWORD, :LPCSTR, :int, :LPWSTR, :int], :int
 
 =begin
 int WideCharToMultiByte(
@@ -127,7 +127,7 @@ int WideCharToMultiByte(
   __out  LPBOOL lpUsedDefaultChar
 );
 =end
-        attach_function :WideCharToMultiByte, [:UINT, :DWORD, :LPCWSTR, :int, :LPSTR, :int, :LPCSTR, :LPBOOL], :int
+        safe_attach_function :WideCharToMultiByte, [:UINT, :DWORD, :LPCWSTR, :int, :LPSTR, :int, :LPCSTR, :LPBOOL], :int
 
         ###############################################
         # Helpers
@@ -139,7 +139,7 @@ int WideCharToMultiByte(
           ustring = (ustring + "").force_encoding('UTF-8') if ustring.respond_to?(:force_encoding) && ustring.encoding.name != "UTF-8"
 
           # ensure we have the double-null termination Windows Wide likes
-          ustring = ustring << "\000\000" if ustring[-1].chr != "\000"
+          ustring = ustring + "\000\000" if ustring[-1].chr != "\000"
 
           # encode it all as UTF-16LE AKA Windows Wide Character AKA Windows Unicode
           ustring = begin

data/lib/chef/win32/error.rb

@@ -22,10 +22,10 @@ require 'chef/win32/unicode'
 require 'chef/exceptions'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     class Error
-      include Chef::Win32::API::Error
-      extend Chef::Win32::API::Error
+      include Chef::ReservedNames::Win32::API::Error
+      extend Chef::ReservedNames::Win32::API::Error
 
       def self.format_message(message_id = 0, args = {})
         flags = args[:flags] || FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_ARGUMENT_ARRAY
@@ -42,7 +42,7 @@ class Chef
         begin
           return buffer.read_pointer.read_wstring(num_chars)
         ensure
-          Chef::Win32::Memory.local_free(buffer.read_pointer)
+          Chef::ReservedNames::Win32::Memory.local_free(buffer.read_pointer)
         end
       end
 

data/lib/chef/win32/file.rb

@@ -22,10 +22,10 @@ require 'chef/win32/api/security'
 require 'chef/win32/error'
 
 class Chef
-  module Win32
+  module ReservedNames::Win32
     class File
-      include Chef::Win32::API::File
-      extend Chef::Win32::API::File
+      include Chef::ReservedNames::Win32::API::File
+      extend Chef::ReservedNames::Win32::API::File
 
       # Creates a symbolic link called +new_name+ for the file or directory
       # +old_name+.
@@ -40,7 +40,7 @@ class Chef
         old_name = encode_path(old_name)
         new_name = encode_path(new_name)
         unless CreateHardLinkW(new_name, old_name, nil)
-          Chef::Win32::Error.raise!
+          Chef::ReservedNames::Win32::Error.raise!
         end
       end
 
@@ -51,14 +51,14 @@ class Chef
       # returns nil as per MRI.
       #
       def self.symlink(old_name, new_name)
-        raise Errno::ENOENT, "(#{old_name}, #{new_name})" unless ::File.exist?(old_name)
+        # raise Errno::ENOENT, "(#{old_name}, #{new_name})" unless ::File.exist?(old_name)
         # TODO do a check for CreateSymbolicLinkW and
         # raise NotImplemented exception on older Windows
         flags = ::File.directory?(old_name) ? SYMBOLIC_LINK_FLAG_DIRECTORY : 0
         old_name = encode_path(old_name)
         new_name = encode_path(new_name)
         unless CreateSymbolicLinkW(new_name, old_name, flags)
-          Chef::Win32::Error.raise!
+          Chef::ReservedNames::Win32::Error.raise!
         end
       end
 
@@ -88,23 +88,73 @@ class Chef
       # will raise a NotImplementedError, as per MRI.
       #
       def self.readlink(link_name)
-        raise Errno::ENOENT, link_name unless ::File.exist?(link_name)
-        # TODO do a check for GetFinalPathNameByHandleW and
-        # raise NotImplemented exception on older Windows
-        file_handle(link_name) do |handle|
-          buffer = FFI::MemoryPointer.new(0.chr * MAX_PATH)
-          num_chars = GetFinalPathNameByHandleW(handle, buffer, buffer.size, FILE_NAME_NORMALIZED)
-          if num_chars == 0
-            Chef::Win32::Error.raise! #could be misleading if problem is too small buffer size as GetLastError won't report failure
+        raise Errno::ENOENT, link_name unless ::File.exists?(link_name)
+        symlink_file_handle(link_name) do |handle|
+          # Go to DeviceIoControl to get the symlink information
+          # http://msdn.microsoft.com/en-us/library/windows/desktop/aa364571(v=vs.85).aspx
+          reparse_buffer = FFI::MemoryPointer.new(MAXIMUM_REPARSE_DATA_BUFFER_SIZE)
+          parsed_size = FFI::Buffer.new(:long).write_long(0)
+          if DeviceIoControl(handle, FSCTL_GET_REPARSE_POINT, nil, 0, reparse_buffer, MAXIMUM_REPARSE_DATA_BUFFER_SIZE, parsed_size, nil) == 0
+            Chef::ReservedNames::Win32::Error.raise!
           end
-          buffer.read_wstring(num_chars).sub(path_prepender, "")
+
+          # Ensure it's a symbolic link
+          reparse_buffer = REPARSE_DATA_BUFFER.new(reparse_buffer)
+          if reparse_buffer[:ReparseTag] != IO_REPARSE_TAG_SYMLINK
+            raise Errno::EACCES, "#{link_name} is not a symlink"
+          end
+
+          # Return the link destination (strip off \??\ at the beginning, which is a local filesystem thing)
+          link_dest = reparse_buffer.reparse_buffer.substitute_name
+          if link_dest =~ /^\\\?\?\\/
+            link_dest = link_dest[4..-1]
+          end
+          link_dest
         end
       end
 
+      # Gets the short form of a path (Administrator -> ADMINI~1)
+      def self.get_short_path_name(path)
+        path = path.to_wstring
+        size = GetShortPathNameW(path, nil, 0)
+        if size == 0
+          Chef::ReservedNames::Win32::Error.raise!
+        end
+        result = FFI::MemoryPointer.new :char, (size+1)*2
+        if GetShortPathNameW(path, result, size+1) == 0
+          Chef::ReservedNames::Win32::Error.raise!
+        end
+        result.read_wstring(size)
+      end
+
+      # Gets the long form of a path (ADMINI~1 -> Administrator)
+      def self.get_long_path_name(path)
+        path = path.to_wstring
+        size = GetLongPathNameW(path, nil, 0)
+        if size == 0
+          Chef::ReservedNames::Win32::Error.raise!
+        end
+        result = FFI::MemoryPointer.new :char, (size+1)*2
+        if GetLongPathNameW(path, result, size+1) == 0
+          Chef::ReservedNames::Win32::Error.raise!
+        end
+        result.read_wstring(size)
+      end
+
       def self.info(file_name)
         Info.new(file_name)
       end
 
+      def self.verify_links_supported!
+        begin
+          CreateSymbolicLinkW(nil)
+        rescue Chef::Exceptions::Win32APIFunctionNotImplemented => e
+          raise e
+        rescue Exception
+          # things are ok.
+        end
+      end
+
       # ::File compat
       class << self
         alias :stat :info