diff options
| author | Sam Roberts <vieuxtech@gmail.com> | 2012-02-21 13:11:39 -0800 |
|---|---|---|
| committer | Sam Roberts <vieuxtech@gmail.com> | 2012-04-11 13:54:01 -0700 |
| commit | 27a3964ff7891a5f240ef08b3294714ef6e28074 (patch) | |
| tree | c5a54c7cb51f643343fba0373f26d3ed7a725941 | |
| parent | db47a91d404d74295213c03cb8b10301fa95f939 (diff) | |
| download | luasocket-27a3964ff7891a5f240ef08b3294714ef6e28074.tar.gz luasocket-27a3964ff7891a5f240ef08b3294714ef6e28074.tar.bz2 luasocket-27a3964ff7891a5f240ef08b3294714ef6e28074.zip | |
socket_strerror() sometimes maps errno, instead of its argument
Looks like a historical bug. Its err argument is an error number, but
if it isn't using a custom error message for it, it just calls
strerror() with the errno global, effectively ignoring its argument
and returning a semi-random string.
| -rw-r--r-- | src/usocket.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/usocket.c b/src/usocket.c index 46087c6..3221cb9 100644 --- a/src/usocket.c +++ b/src/usocket.c | |||
| @@ -418,8 +418,8 @@ const char *socket_strerror(int err) { | |||
| 418 | case ECONNREFUSED: return "connection refused"; | 418 | case ECONNREFUSED: return "connection refused"; |
| 419 | case ECONNABORTED: return "closed"; | 419 | case ECONNABORTED: return "closed"; |
| 420 | case ECONNRESET: return "closed"; | 420 | case ECONNRESET: return "closed"; |
| 421 | case ETIMEDOUT: return "timedout"; | 421 | case ETIMEDOUT: return "timeout"; |
| 422 | default: return strerror(errno); | 422 | default: return strerror(err); |
| 423 | } | 423 | } |
| 424 | } | 424 | } |
| 425 | 425 | ||