exec_sandbox 0.1.1 → 0.1.2

data/VERSION

@@ -1 +1 @@
-0.1.1
+0.1.2

data/exec_sandbox.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "exec_sandbox"
-  s.version = "0.1.1"
+  s.version = "0.1.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Victor Costan"]

data/lib/exec_sandbox/sandbox.rb

@@ -118,10 +118,15 @@ class Sandbox
     out_wr.close if out_wr
     
     # Collect information about the child.
+    if out_rd
+      out_pieces = []
+      out_pieces << out_rd.read rescue nil
+    end
     status = ExecSandbox::Wait4.wait4 pid
     if out_rd
-      status[:out_data] = out_rd.read
+      out_pieces << out_rd.read rescue nil
       out_rd.close
+      status[:out_data] = out_pieces.join('')
     end
     status
   end

data/spec/exec_sandbox/sandbox_spec.rb

@@ -29,18 +29,37 @@ describe ExecSandbox::Sandbox do
   end
   
   describe 'pipe redirection' do
-    before do
-      ExecSandbox.use do |s|
-        @result = s.run bin_fixture(:duplicate), :in_data => "Pipe test\n"
+    describe 'duplicate.rb' do
+      before do
+        ExecSandbox.use do |s|
+          @result = s.run bin_fixture(:duplicate), :in_data => "Pipe test\n"
+        end
+      end
+      
+      it 'should not crash' do
+        @result[:exit_code].should == 0
+      end
+      
+      it 'should produce the correct result' do
+        @result[:out_data].should == "Pipe test\nPipe test\n"
       end
     end
     
-    it 'should not crash' do
-      @result[:exit_code].should == 0
-    end
-    
-    it 'should produce the correct result' do
-      @result[:out_data].should == "Pipe test\nPipe test\n"
+    describe 'buffer.rb' do
+      let(:buffer_size) { 1024 * 1024 }
+      before do
+        ExecSandbox.use do |s|
+          @result = s.run [bin_fixture(:buffer), '', buffer_size.to_s]
+        end
+      end
+      
+      it 'should not crash' do
+        @result[:exit_code].should == 0
+      end
+      
+      it 'should produce the correct result' do
+        @result[:out_data].should == "S" * buffer_size
+      end
     end
   end
   

data/spec/fixtures/buffer.rb

@@ -3,8 +3,13 @@
 # Allocates a buffer sized according to the second argument and writes it to the
 # file indicated by the first argument.
 
-buffer = String.new "\xFE" * ARGV[1].to_i
-File.open(ARGV[0], 'wb') do |f|
-  f.write buffer
-  f.flush
+buffer = String.new "S" * ARGV[1].to_i
+if ARGV[0].empty?
+  STDOUT.write buffer
+  STDOUT.flush
+else
+  File.open(ARGV[0], 'wb') do |f|
+    f.write buffer
+    f.flush
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: exec_sandbox
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-10-13 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
-  requirement: &22094080 !ruby/object:Gem::Requirement
+  requirement: &24549880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 1.0.9
   type: :runtime
   prerelease: false
-  version_requirements: *22094080
+  version_requirements: *24549880
 - !ruby/object:Gem::Dependency
   name: rdoc
-  requirement: &22093480 !ruby/object:Gem::Requirement
+  requirement: &24549360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '3.10'
   type: :development
   prerelease: false
-  version_requirements: *22093480
+  version_requirements: *24549360
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &22092840 !ruby/object:Gem::Requirement
+  requirement: &24548820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *22092840
+  version_requirements: *24548820
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &22092240 !ruby/object:Gem::Requirement
+  requirement: &24548320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: 0.7.2
   type: :development
   prerelease: false
-  version_requirements: *22092240
+  version_requirements: *24548320
 - !ruby/object:Gem::Dependency
   name: yard-rspec
-  requirement: &22091600 !ruby/object:Gem::Requirement
+  requirement: &24547800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0.1'
   type: :development
   prerelease: false
-  version_requirements: *22091600
+  version_requirements: *24547800
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &22090920 !ruby/object:Gem::Requirement
+  requirement: &24547220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: 1.0.21
   type: :development
   prerelease: false
-  version_requirements: *22090920
+  version_requirements: *24547220
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &22089700 !ruby/object:Gem::Requirement
+  requirement: &24546700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *22089700
+  version_requirements: *24546700
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &22089180 !ruby/object:Gem::Requirement
+  requirement: &24546160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *22089180
+  version_requirements: *24546160
 description: Temporary users and groups, rlimits
 email: costan@gmail.com
 executables: []
@@ -150,7 +150,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1718344234877287063
+      hash: -4313873554503945480
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: