diff options
author | Lukas Larsson <[email protected]> | 2014-09-12 12:06:34 +0200 |
---|---|---|
committer | Lukas Larsson <[email protected]> | 2014-09-12 12:06:34 +0200 |
commit | a5bfb0a58783c472887ce0ad2060e4a395aa941d (patch) | |
tree | 93dd76737309acb93b79d58a6079ddf15833c5a1 /erts/emulator/drivers/common/inet_drv.c | |
parent | daa9e1a0d97612f6689685e5a597375ceb9e7b81 (diff) | |
download | otp-a5bfb0a58783c472887ce0ad2060e4a395aa941d.tar.gz otp-a5bfb0a58783c472887ce0ad2060e4a395aa941d.tar.bz2 otp-a5bfb0a58783c472887ce0ad2060e4a395aa941d.zip |
erts: getsockname is not allowed on non-bound sockets
This only produces an error on win32, but should not really be called
on *nix either.
Diffstat (limited to 'erts/emulator/drivers/common/inet_drv.c')
-rw-r--r-- | erts/emulator/drivers/common/inet_drv.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/erts/emulator/drivers/common/inet_drv.c b/erts/emulator/drivers/common/inet_drv.c index 891589d1c5..db8a251fdd 100644 --- a/erts/emulator/drivers/common/inet_drv.c +++ b/erts/emulator/drivers/common/inet_drv.c @@ -4542,11 +4542,13 @@ static ErlDrvSSizeT inet_ctl_fdopen(inet_descriptor* desc, int domain, int type, inet_address name; unsigned int sz = sizeof(name); - /* check that it is a socket and that the socket is bound */ - if (IS_SOCKET_ERROR(sock_name(s, (struct sockaddr*) &name, &sz))) - return ctl_error(sock_errno(), rbuf, rsize); - if (name.sa.sa_family != domain) - return ctl_error(EINVAL, rbuf, rsize); + if (bound) { + /* check that it is a socket and that the socket is bound */ + if (IS_SOCKET_ERROR(sock_name(s, (struct sockaddr*) &name, &sz))) + return ctl_error(sock_errno(), rbuf, rsize); + if (name.sa.sa_family != domain) + return ctl_error(EINVAL, rbuf, rsize); + } #ifdef __OSE__ /* for fdopen duplicating the sd will allow to uniquely identify the signal from OSE with erlang port */ |