bugfix: higher mode bits on local connection

inspec · Jul 10, 2016 · d028748 · d028748
1 parent f64da2e
commit d028748
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 1 deletion.
diff --git a/lib/train/transports/local_file.rb b/lib/train/transports/local_file.rb
@@ -69,7 +69,7 @@ def stat
 
       @stat = {
         type: Train::Extras::Stat.find_type(file_stat.mode),
-        mode: file_stat.mode & 00777,
+        mode: file_stat.mode & 07777,
         mtime: file_stat.mtime.to_i,
         size: file_stat.size,
         owner: pw_username(file_stat.uid),

diff --git a/test/integration/bootstrap.sh b/test/integration/bootstrap.sh
@@ -9,6 +9,9 @@ test ! -e /tmp/symlink && \
 chmod 0777 /tmp/symlink
 chmod 0765 /tmp/file
 
+echo -n 'hello suid/sgid/sticky' > /tmp/sfile
+chmod 7765 /tmp/sfile
+
 test ! -e /tmp/pipe && \
   mkfifo /tmp/pipe
 

diff --git a/test/integration/cookbooks/test/recipes/prep_files.rb b/test/integration/cookbooks/test/recipes/prep_files.rb
@@ -16,6 +16,13 @@
   content 'hello world'
 end
 
+file '/tmp/sfile' do
+  mode '7765'
+  owner 'root'
+  group gid
+  content 'hello suid/sgid/sticky'
+end
+
 directory '/tmp/folder' do
   mode '0567'
   owner 'root'

diff --git a/test/integration/tests/path_file_test.rb b/test/integration/tests/path_file_test.rb
@@ -82,4 +82,11 @@
       j['type'].must_equal :file
     end
   end
+
+  describe 'regular file' do
+    let(:file) { backend.file('/tmp/sfile') }
+    it 'has mode 7765' do
+      file.mode.must_equal(07765)
+    end
+  end
 end