DRb mixing up function return values between PIDs after fork()
The precise details are in the attached test case, but the basic version goes as such: Child pid 1 calls method_a() but gets the return value of method_b() that was called from child pid 2. This only happens if the parent process makes any remote DRb calls before the fork.
This issue showed up for me in 1.9.1 as well as 1.8.7 on x86, but not in x86_64.
Updated by mame (Yusuke Endoh) almost 11 years ago
- Status changed from Open to Assigned
- Assignee set to seki (Masatoshi Seki)
Sorry for late response, but I cannot reproduce on both 1.8.7-p249
I assign this issue to Masatoshi Seki, the maintainer for drb.
Seki-san, could you check this issue?
Yusuke Endoh firstname.lastname@example.org
Updated by rhythmx (Sean Bradly) over 10 years ago
I just verified that this shows up for me in Ruby 1.8.7-p249 as well as 1.9.1-p378. It does seem to reproduce a lot more readily in 1.8, however.
Note that you must run the test script with at least one argument (the arg can be anything) to trigger the problem. With 0 arguments it skips the bug to show the known good case.
(o.0) xi:~/code $ rvm list
ruby-1.8.6-p383 [ i386 ] ruby-1.8.7-p174 [ i386 ] ruby-1.8.7-p249 [ i386 ]
=> ruby-1.9.1-p378 [ i386 ]
(o.0) xi:~/code $ ruby ~/Downloads/drb_test_case.rb FAIL
A BAD RESULT ("BBBB")
a exitinga exiting
Updated by jeremyevans (Jeremy Evans) over 1 year ago
- Status changed from Assigned to Closed
Applied in changeset git|d0ed935d5bf8c3fce9800742a36e44fb7f63dda4.
Fix some DRb issues (#2552)
- Handle BasicObject in drb
Also fix a bug in rescue clause of any_to_s because sprintf
does not handle the %l modifier.
Fixes [Bug #7833]
- Do not send a reply to the client if there is a connection error
This allows for normal TCP shutdown (fin-ack-fin-ack instead of
Patch from email@example.com (Pierre-Alexandre Meyer).
Fixes [Bug #2339]
- Detect fork and do not reuse forked connections in drb
This associates each DRbConn with a pid, and if the pid changes,
it closes any DRbConns in the pool with a pid that no longer
matches. This fixes DRb servers from sending messages intended
for one client to another client after forking.