Project

General

Profile

Actions

Bug #17813

closed

rb_funcall() may reset the latest socket error unexpectedlly since Ruby 3.0.0

Added by kenhys (Kentaro Hayashi) almost 3 years ago. Updated almost 2 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 3.0.0p0 (2020-12-25 revision 95aff21468) [x64-mingw32]
[ruby-core:103511]

Description

The problem

It seems that when rb_funcall() is called, the socket error is reset.
This behavior is introduced since Ruby 3.0.0 on Windows.

With this incompatible change, it fails to get WSAGetLastError correctly
even though the previous funciton call fails.

Does it intentional change?

The actual situation

This incompatible behavior is found with the following Fiddle example.

module WinSock
  require 'fiddle/import'
  extend Fiddle::Importer
  dlload 'ws2_32.dll'
  extern 'int bind(int, void *, int)'
  extern 'int WSAGetLastError(void)'
end

p WinSock.bind(0, nil, 0)
p WinSock.WSAGetLastError

With Ruby 3.0 (ruby 3.0.0p0 (2020-12-25 revision 95aff21468) [x64-mingw32]),
WinSock.bind(0, nil, 0) should be failed, thus it returns -1. (expected)
WinSock.WSAGetLastError should return 10038 (WSAENOTSOCK) in this case, but returns 0. (unexpected)

With Ruby 2.7 (ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x64-mingw32]),
WinSock.bind(0, nil, 0) returns -1, (expected)
WinSock.WSAGetLastError return 10038 (WSAENOTSOCK). (expected)

Thus, the use case such as checking bind result and report socket
related error via (WSAGetLastError) doesn't work as expected with Ruby
3.0 (Windows).

The expected status

If this change is introduced by intendedly, no need to change ruby itself.
If not, it may be better to be fixed.

Additional Information

I've reported this issue to Fiddle, then since Fiddle 1.0.8,
Fiddle.win32_last_socket_error is implemented as a workaround.

Unexpected error code with WSAGetLastError even though the previous function calll was failed
https://github.com/ruby/fiddle/issues/72

Updated by usa (Usaku NAKAMURA) over 2 years ago

Ruby cannot guarantee that the state of WSA does not change between two method calls, because select is called internally when thread switch occurs.
So, the solution of fiddle (adding Fiddle.win32_last_socket_error) is unavoidable in this case, I guess.

Aaron, the maintainer of Ruby's fiddle, can you backport fiddle from upstream?

Updated by mame (Yusuke Endoh) almost 2 years ago

  • Status changed from Open to Closed

This ticket was created to confirm whether this is a bug of Ruby (https://github.com/ruby/fiddle/issues/72#issuecomment-822253220), and @usa (Usaku NAKAMURA) answered that it was not. And the fix in fiddle has been already backported to ruby/ruby 71d4a493b890a21fdf3b302849d6d60c11ba1d9e. I believe there is nothing to do in this ticket. Closing.

Actions

Also available in: Atom PDF

Like0
Like0Like0