Backport #7358

Wrong fd redirection on fork

Added by Felipe Contreras over 1 year ago. Updated 7 months ago.

[ruby-core:49370]
Status:Assigned
Priority:Normal
Assignee:Usaku NAKAMURA

Description

It seems fd redirecton only works with spawn, not with fork+exec (at least for fd 3).

% ruby -v
ruby 1.9.3p327 (2012-11-10 revision 37606) [x86_64-linux]

#!/usr/bin/env ruby

tmp = File.new('/tmp/foo', File::CREAT | File::WRONLY)
cmd = ['echo', 'yes']

pid = Process.spawn(*cmd, 3 => tmp)
Process.wait(pid)

pid = fork do
Process.exec(*cmd, :close_others => true, 3 => tmp)
end
Process.wait(pid)

History

#1 Updated by Eric Hodel over 1 year ago

  • Status changed from Open to Rejected

=begin
If you want data to show up in ((%/tmp/foo%)), redirect stdout in the child process. Setting file descriptor 3 does nothing since ((%echo%)) makes no use of it.

#!/usr/bin/env ruby

tmp = File.new('/tmp/foo', File::CREAT | File::WRONLY)
cmd = ['echo', 'yes']

pid = Process.spawn(*cmd, out: tmp)
Process.wait(pid)

pid = fork do
Process.exec(*cmd, close_others: true, out: tmp)
end
Process.wait(pid)
=end

#2 Updated by Felipe Contreras 8 months ago

That's not what I want, and yes, setting that does nothing, or should do nothing, but that's not what the command above did, it failed.

Fortunately it seems to be fixed now.

#3 Updated by Felipe Contreras 7 months ago

felipec (Felipe Contreras) wrote:

That's not what I want, and yes, setting that does nothing, or should do nothing, but that's not what the command above did, it failed.

Fortunately it seems to be fixed now.

Actually, I went down to bisect where the problem was fixed, and I found it:

https://github.com/ruby/ruby/commit/2fb032b74be9f306fbd2f95d85f8352f4a6027d6

So it turns out it's fixed in Ruby v2.0, but not v1.9, so this is not "rejected", it should be "already fixed in v2.0", and maybe you would want to backport the fix.

FTR. This is the error I get before the patch:

yes
[ASYNC BUG] consumecommunicationpipe: read

EBADF

ruby 2.0.0dev (2012-06-09 trunk 35968) [x86_64-linux]

[NOTE]
You may have encountered a bug in the Ruby interpreter or extension libraries.
Bug reports are welcome.
For details: http://www.ruby-lang.org/bugreport.html

#4 Updated by Nobuyoshi Nakada 7 months ago

  • Tracker changed from Bug to Backport
  • Project changed from ruby-trunk to Backport93
  • Status changed from Rejected to Assigned
  • Assignee set to Usaku NAKAMURA

Also available in: Atom PDF