Socket.st
changeset 3126 d95cc55a4fb5
parent 3123 55ef9ca4f09f
child 3127 333e87ba225a
--- a/Socket.st	Sat Oct 12 09:08:40 2013 +0200
+++ b/Socket.st	Mon Oct 14 15:29:32 2013 +0200
@@ -1896,7 +1896,6 @@
 # ifdef SO_REUSEADDR
     if (reuse == true) {
         int on = 1;
-
         if (setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, (char *) &on, sizeof (on)) < 0) {
             DBGPRINTF(("SOCKET: setsockopt - SO_REUSEADDR failed\n"));
         }
@@ -1909,6 +1908,9 @@
         __threadErrno = 0;
         ret = STX_WSA_NOINT_CALL3("bind", bind, sock, &sa, sockaddr_size);
     } while ((ret < 0) && (__threadErrno == EINTR));
+    if (ret < 0) {
+        errno = __threadErrno;
+    }
 #  else
     __BEGIN_INTERRUPTABLE__
     do {
@@ -1917,9 +1919,15 @@
     __END_INTERRUPTABLE__
 #  endif
 # else
+    errno = 0;
     ret = bind(sock, (struct sockaddr *)&sa, sockaddr_size);
 # endif
     if (ret < 0) {
+# ifdef WIN32
+        if (errno == 0) {
+            errno = WSAGetLastError();
+        }
+# endif
         DBGPRINTF(("SOCKET: bind failed errno=%d\n", errno));
         error = __INST(lastErrorNumber) = __MKSMALLINT(errno);
         goto getOutOfHere;
@@ -1949,11 +1957,11 @@
 
     "
      (Socket domain:#'AF_INET' type:#stream)
-        bindTo:(IPSocketAddress anyHost port:2122) reuseAddress:false;
+        bindTo:(IPSocketAddress anyHost port:445) reuseAddress:false;
         yourself.
 
      (Socket domain:#'AF_INET' type:#stream)
-        bindTo:nil reuseAddress:false;
+        bindTo:139 reuseAddress:false;
         yourself.
 
      (Socket domain:#'AF_INET6' type:#stream)
@@ -1975,7 +1983,7 @@
      aNumber is the number of connect requests, that may be queued on the socket"
 
     handle isNil ifTrue:[
-	^ self errorNotOpen
+        ^ self errorNotOpen
     ].
 %{
 #ifndef NO_SOCKET
@@ -1984,8 +1992,8 @@
     int ret;
 
     if (! __isSmallInteger(aNumber)) {
-	DBGPRINTF(("SOCKET: invalid arg\n"));
-	RETURN (false);
+        DBGPRINTF(("SOCKET: invalid arg\n"));
+        RETURN (false);
     }
 
     sock = SOCKET_FROM_FILE_OBJECT(fp);
@@ -1993,24 +2001,33 @@
 #ifdef LISTEN_BLOCKS
 # ifdef DO_WRAP_CALLS
     do {
-	__threadErrno = 0;
-	ret = STX_WSA_NOINT_CALL2("listen", listen, sock, __intVal(aNumber));
+        __threadErrno = 0;
+        ret = STX_WSA_NOINT_CALL2("listen", listen, sock, __intVal(aNumber));
     } while ((ret < 0) && (__threadErrno == EINTR));
+    if (ret < 0) {
+        errno = __threadErrno;
+    }
 # else
     __BEGIN_INTERRUPTABLE__
     do {
-	ret = listen(sock, __intVal(aNumber));
+        ret = listen(sock, __intVal(aNumber));
     } while ((ret < 0) && (errno == EINTR));
     __END_INTERRUPTABLE__
 # endif
 #else
+    errno = 0;
     ret = listen(sock, __intVal(aNumber));
 #endif
 
     if (ret < 0) {
-	DBGPRINTF(("SOCKET: listen call failed errno=%d\n", errno));
-	__INST(lastErrorNumber) = __MKSMALLINT(errno);
-	RETURN (false);
+# ifdef WIN32
+        if (errno == 0) {
+            errno = WSAGetLastError();
+        }
+# endif
+        DBGPRINTF(("SOCKET: listen call failed errno=%d\n", errno));
+        __INST(lastErrorNumber) = __MKSMALLINT(errno);
+        RETURN (false);
     }
 #else
     RETURN (false);
@@ -2310,74 +2327,76 @@
      The thread blocks until data arrives - you may want to wait before
      receiving, using #readWait or #readWaitWithTimeout:."
 
-    |nReceived|
+    |nReceived error|
 
 %{
 #ifndef NO_SOCKET
     OBJ fp = __INST(handle);
 
     if (fp != nil) {
-	SOCKET sock;
-	INT objSize, offs;
-	INT n;
-	char *extPtr;
-	unsigned char *buffer;
-	unsigned char *allocatedBuffer = NULL;
-	INT flags = 0;
-
-	sock = SOCKET_FROM_FILE_OBJECT(fp);
-
-	if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
-	if (__isSmallInteger(nBytes)) {
-	    if (__intVal(nBytes) < objSize) {
-		objSize = __intVal(nBytes);
-	    }
-	}
+        SOCKET sock;
+        INT objSize, offs;
+        INT n;
+        char *extPtr;
+        unsigned char *buffer;
+        unsigned char *allocatedBuffer = NULL;
+        INT flags = 0;
+
+        sock = SOCKET_FROM_FILE_OBJECT(fp);
+
+        if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
+        if (__isSmallInteger(nBytes)) {
+            if (__intVal(nBytes) < objSize) {
+                objSize = __intVal(nBytes);
+            }
+        }
 
 # ifdef DO_WRAP_CALLS
-	if (extPtr) {
-	    buffer = extPtr + offs;
-	} else {
-	    allocatedBuffer = buffer = (char *)malloc(objSize);
-	}
-
-	do {
-	    __threadErrno = 0;
-	    n = (INT)STX_WSA_NOINT_CALL4("recv", recv, sock, buffer, objSize, flags);
-	} while ((n < 0) && (__threadErrno == EINTR));
-
-	if (allocatedBuffer) {
-	    if (n > 0) {
-		bcopy(allocatedBuffer, (char *)__InstPtr(aDataBuffer) + offs, n);
-	    }
-	    free(allocatedBuffer);
-	}
+        if (extPtr) {
+            buffer = extPtr + offs;
+        } else {
+            allocatedBuffer = buffer = (char *)malloc(objSize);
+        }
+
+        do {
+            __threadErrno = 0;
+            n = (INT)STX_WSA_NOINT_CALL4("recv", recv, sock, buffer, objSize, flags);
+        } while ((n < 0) && (__threadErrno == EINTR));
+        if (n < 0) {
+            errno = __threadErrno;
+        }
+        
+        if (allocatedBuffer) {
+            if (n > 0) {
+                bcopy(allocatedBuffer, (char *)__InstPtr(aDataBuffer) + offs, n);
+            }
+            free(allocatedBuffer);
+        }
 # else
-	__BEGIN_INTERRUPTABLE__
-	do {
-	    if (extPtr) {
-		n = recv(sock, extPtr + offs, objSize, flags);
-	    } else {
-		n = recv(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, flags);
-	    }
-	} while ((n < 0) && (errno == EINTR));
-	__END_INTERRUPTABLE__
+        __BEGIN_INTERRUPTABLE__
+        do {
+            if (extPtr) {
+                n = recv(sock, extPtr + offs, objSize, flags);
+            } else {
+                n = recv(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, flags);
+            }
+        } while ((n < 0) && (errno == EINTR));
+        __END_INTERRUPTABLE__
 # endif
 
-	if (n < 0) {
-	    __INST(lastErrorNumber) = __MKSMALLINT(errno);
-	}
-	nReceived = __MKSMALLINT(n);
+        if (n < 0) {
+            error = __INST(lastErrorNumber) = __MKSMALLINT(errno);
+        } else {
+            RETURN(__MKSMALLINT(n));
+        }        
     }
 #endif
 bad: ;
 %}.
-    nReceived notNil ifTrue:[
-	nReceived < 0 ifTrue:[
-	    'Socket [warning]: ' infoPrint.
-	    (OperatingSystem errorTextForNumber:lastErrorNumber) infoPrintCR.
-	].
-	^ nReceived
+    error notNil ifTrue:[
+        ^ ReadError
+            raiseRequestWith:self
+            errorString:(' : ' , OperatingSystem errorTextForNumber:error)
     ].
     "
      arrive here if you try to receive into an invalid buffer (i.e. not ByteArray-like)
@@ -2409,21 +2428,21 @@
      The thread blocks until data arrives - you may want to wait before
      receiving, using #readWait or #readWaitWithTimeout:."
 
-    |domainClass addr addrLen nReceived|
+    |domainClass addr addrLen nReceived error|
 
     domainClass := self class socketAddressClassForDomain:domain.
     domainClass isNil ifTrue:[
-	^ self error:'invalid (unsupported) domain'.
+        ^ self error:'invalid (unsupported) domain'.
     ].
     (anAddressBuffer isKindOf:SocketAddress) ifTrue:[
-	anAddressBuffer class == domainClass ifFalse:[
-	    ^ self error:'addressBuffer class mismatch (domain)'.
-	].
-	addr := anAddressBuffer.
+        anAddressBuffer class == domainClass ifFalse:[
+            ^ self error:'addressBuffer class mismatch (domain)'.
+        ].
+        addr := anAddressBuffer.
     ] ifFalse:[
-	anAddressBuffer notNil ifTrue:[
-	    addr := domainClass new.
-	].
+        anAddressBuffer notNil ifTrue:[
+            addr := domainClass new.
+        ].
     ].
 
 %{
@@ -2431,101 +2450,106 @@
     OBJ fp = __INST(handle);
 
     if (fp != nil) {
-	SOCKET sock;
-	INT objSize;
-	union sockaddr_u sa;
-	unsigned INT alen = 0;
-	INT n, offs;
-	INT flags = 0;
-	char *extPtr;
-	unsigned char *allocatedBuffer = NULL;
-	unsigned char *buffer = NULL;
-
-	sock = SOCKET_FROM_FILE_OBJECT(fp);
-
-	if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
-	if (__isSmallInteger(nBytes)) {
-	    if (__intVal(nBytes) < objSize) {
-		objSize = __intVal(nBytes);
-	    }
-	}
+        SOCKET sock;
+        INT objSize;
+        union sockaddr_u sa;
+        unsigned INT alen = 0;
+        INT n, offs;
+        INT flags = 0;
+        char *extPtr;
+        unsigned char *allocatedBuffer = NULL;
+        unsigned char *buffer = NULL;
+
+        sock = SOCKET_FROM_FILE_OBJECT(fp);
+
+        if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
+        if (__isSmallInteger(nBytes)) {
+            if (__intVal(nBytes) < objSize) {
+                objSize = __intVal(nBytes);
+            }
+        }
 # ifdef DO_WRAP_CALLS
-	if (extPtr) {
-	    buffer = extPtr + offs;
-	} else {
-	    allocatedBuffer = buffer = (char *)malloc(objSize);
-	}
-
-	do {
-	    __threadErrno = 0;
-	    alen = sizeof(sa);
-	    n = (INT)STX_WSA_NOINT_CALL6("recvfrom", recvfrom, sock, buffer, objSize, flags, (struct sockaddr *)&sa, &alen);
-	} while ((n < 0) && (__threadErrno == EINTR));
-
-	if (allocatedBuffer) {
-	    if (n > 0) {
-		bcopy(allocatedBuffer, (char *)__InstPtr(aDataBuffer) + offs, n);
-	    }
-	    free(allocatedBuffer);
-	}
+        if (extPtr) {
+            buffer = extPtr + offs;
+        } else {
+            allocatedBuffer = buffer = (char *)malloc(objSize);
+        }
+
+        do {
+            __threadErrno = 0;
+            alen = sizeof(sa);
+            n = (INT)STX_WSA_NOINT_CALL6("recvfrom", recvfrom, sock, buffer, objSize, flags, (struct sockaddr *)&sa, &alen);
+        } while ((n < 0) && (__threadErrno == EINTR));
+        if (n < 0) {
+            errno = __threadErrno;
+        }
+
+        if (allocatedBuffer) {
+            if (n > 0) {
+                memcpy((char *)__InstPtr(aDataBuffer) + offs, allocatedBuffer, n);
+            }
+            free(allocatedBuffer);
+        }
 # else
-	__BEGIN_INTERRUPTABLE__
-	do {
-	    alen = sizeof(sa);
-	    if (extPtr) {
-		n = recvfrom(sock, extPtr + offs, objSize, flags, (struct sockaddr *) &sa, &alen);
-	    } else {
-		n = recvfrom(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, flags, (struct sockaddr *) &sa, &alen);
-	    }
-	} while ((n < 0) && (errno == EINTR));
-	__END_INTERRUPTABLE__
+        __BEGIN_INTERRUPTABLE__
+        do {
+            alen = sizeof(sa);
+            if (extPtr) {
+                n = recvfrom(sock, extPtr + offs, objSize, flags, (struct sockaddr *) &sa, &alen);
+            } else {
+                n = recvfrom(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, flags, (struct sockaddr *) &sa, &alen);
+            }
+        } while ((n < 0) && (errno == EINTR));
+        __END_INTERRUPTABLE__
 # endif
 
-	if (n >= 0) {
-	    if (__isNonNilObject(addr)) {
-		char *addrPtr;
-		OBJ oClass;
-		int nInstVars, nInstBytes, objSize;
-
-		oClass = __qClass(addr);
-		if (! __isBytes(addr) )
-		    goto bad;
-		nInstVars = __intVal(__ClassInstPtr(oClass)->c_ninstvars);
-		nInstBytes = OHDR_SIZE + (nInstVars * sizeof(OBJ));
-		objSize = __qSize(addr) - nInstBytes;
-		addrPtr = (char *)__InstPtr(addr) + nInstBytes;
-		if (objSize < alen)
-		    goto bad;
-
-		/*
-		 * extract the datagrams address
-		 */
-		bcopy((char *)&sa, addrPtr, alen);
-		addrLen = __MKSMALLINT(alen);
-	    }
-	}
-	if (n < 0) {
-	    __INST(lastErrorNumber) = __MKSMALLINT(errno);
-	}
-	nReceived = __MKSMALLINT(n);
+        if (n >= 0) {
+            if (__isNonNilObject(addr)) {
+                char *addrPtr;
+                OBJ oClass;
+                int nInstVars, nInstBytes, objSize;
+
+                oClass = __qClass(addr);
+                if (! __isBytes(addr) )
+                    goto bad;
+                nInstVars = __intVal(__ClassInstPtr(oClass)->c_ninstvars);
+                nInstBytes = OHDR_SIZE + (nInstVars * sizeof(OBJ));
+                objSize = __qSize(addr) - nInstBytes;
+                addrPtr = (char *)__InstPtr(addr) + nInstBytes;
+                if (objSize < alen)
+                    goto bad;
+
+                /*
+                 * extract the datagrams address
+                 */
+                memcpy(addrPtr, (char *)&sa, alen);
+                addrLen = __MKSMALLINT(alen);
+            }
+        }
+        if (n < 0) {
+            error = __INST(lastErrorNumber) = __MKSMALLINT(errno);
+        }
+        nReceived = __MKSMALLINT(n);
     }
 #endif
 bad: ;
 %}.
+    error notNil ifTrue:[
+        ^ ReadError
+            raiseRequestWith:self
+            errorString:(' : ' , OperatingSystem errorTextForNumber:error)
+    ].
+
     nReceived notNil ifTrue:[
-	nReceived < 0 ifTrue:[
-	    'Socket [warning]: ' infoPrint.
-	    (OperatingSystem errorTextForNumber:lastErrorNumber) infoPrintCR.
-	].
-	addrLen notNil ifTrue:[
-	    (addr == anAddressBuffer) ifFalse:[
-		self obsoleteFeatureWarning:'please use a socketAddress argument'.
-
-		" can be a ByteArray for backward compatibility "
-		anAddressBuffer replaceFrom:1 to:addrLen with:(addr hostAddress).
-	    ].
-	].
-	^ nReceived
+        addrLen notNil ifTrue:[
+            (addr == anAddressBuffer) ifFalse:[
+                self obsoleteFeatureWarning:'please use a socketAddress argument'.
+
+                "can be a ByteArray for backward compatibility"
+                anAddressBuffer replaceFrom:1 to:addrLen with:(addr hostAddress).
+            ].
+        ].
+        ^ nReceived
     ].
     "
      arrive here if you try to receive into an invalid buffer
@@ -2544,7 +2568,7 @@
      On error, the unix error code is left in the lastErrorNumber
      instance variable."
 
-    |nReceived portNo|
+    |error|
 
 %{
 #ifndef NO_SOCKET
@@ -2553,67 +2577,77 @@
     if ((fp != nil)
      && __isSmallInteger(startIndex)
      && __isSmallInteger(nBytes)) {
-	SOCKET sock;
-	INT objSize;
-	INT n;
-	char *extPtr;
-	INT _flags = 0;
-	INT offs;
-	unsigned long norder;
-	unsigned char *buffer;
-	unsigned char *allocatedBuffer = NULL;
-
-	_flags = __longIntVal(flags);
-
-	sock = SOCKET_FROM_FILE_OBJECT(fp);
-
-	if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
-	if (__isSmallInteger(nBytes)) {
-	    if (__intVal(nBytes) < objSize) {
-		objSize = __intVal(nBytes);
-	    }
-	}
+        SOCKET sock;
+        INT objSize;
+        INT n;
+        char *extPtr;
+        INT _flags = 0;
+        INT offs;
+        unsigned long norder;
+        unsigned char *buffer;
+        unsigned char *allocatedBuffer = NULL;
+
+        _flags = __longIntVal(flags);
+
+        sock = SOCKET_FROM_FILE_OBJECT(fp);
+
+        if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
+        if (__isSmallInteger(nBytes)) {
+            if (__intVal(nBytes) < objSize) {
+                objSize = __intVal(nBytes);
+            }
+        }
 
 # ifdef DGRAM_DEBUG
-	console_printf("sending %d bytes ...\n", nBytes);
+        console_printf("sending %d bytes ...\n", nBytes);
 # endif
 
 #ifdef DO_WRAP_CALLS
-	if (extPtr) {
-	    buffer = extPtr + offs;
-	} else {
-	    allocatedBuffer = buffer = (char *)malloc(objSize);
-	    bcopy((char *)__InstPtr(aDataBuffer) + offs, allocatedBuffer, objSize);
-	}
-
-	do {
-	    __threadErrno = 0;
-	    n = (INT)STX_WSA_NOINT_CALL4("send", send, sock, buffer, objSize, _flags);
-	} while ((n < 0) && (__threadErrno == EINTR));
-
-	if (allocatedBuffer) {
-	    free(allocatedBuffer);
-	}
+        if (extPtr) {
+            buffer = extPtr + offs;
+        } else {
+            allocatedBuffer = buffer = (char *)malloc(objSize);
+            bcopy((char *)__InstPtr(aDataBuffer) + offs, allocatedBuffer, objSize);
+        }
+
+        do {
+            __threadErrno = 0;
+            n = (INT)STX_WSA_NOINT_CALL4("send", send, sock, buffer, objSize, _flags);
+        } while ((n < 0) && (__threadErrno == EINTR));
+        if (n < 0) {
+            errno = __threadErrno;
+        }
+
+        if (allocatedBuffer) {
+            free(allocatedBuffer);
+        }
 #else
-	__BEGIN_INTERRUPTABLE__
-	do {
-	    if (extPtr) {
-		n = send(sock, extPtr + offs, objSize, _flags);
-	    } else {
-		n = send(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, _flags);
-	    }
-	} while ((n < 0) && (errno == EINTR));
-	__END_INTERRUPTABLE__
+        __BEGIN_INTERRUPTABLE__
+        do {
+            if (extPtr) {
+                n = send(sock, extPtr + offs, objSize, _flags);
+            } else {
+                n = send(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, _flags);
+            }
+        } while ((n < 0) && (errno == EINTR));
+        __END_INTERRUPTABLE__
 #endif
 
-	if (n < 0) {
-	    __INST(lastErrorNumber) = __MKSMALLINT(errno);
-	}
-	RETURN (__MKSMALLINT(n));
+        if (n < 0) {
+            error = __INST(lastErrorNumber) = __MKSMALLINT(errno);
+        } else {
+            RETURN (__MKSMALLINT(n));
+        }            
     }
 #endif
 bad: ;
 %}.
+    error notNil ifTrue:[
+        WriteError
+            raiseRequestWith:self
+            errorString:(' : ' , OperatingSystem errorTextForNumber:error)
+    ].
+
     "
      arrive here if you try to send from an invalid buffer (i.e. not ByteArray-like),
     "
@@ -2656,19 +2690,19 @@
      On error, the unix error code is left in the lastErrorNumber
      instance variable."
 
-    |domainClass addr|
+    |domainClass addr error|
 
     (anAddressBuffer isKindOf:SocketAddress) ifTrue:[
-	addr := anAddressBuffer.
+        addr := anAddressBuffer.
     ] ifFalse:[
-	anAddressBuffer isByteArray ifFalse:[
-	    ^ self error:'bad socketAddress argument'
-	].
-	domainClass := self class socketAddressClassForDomain:domain.
-	domainClass isNil ifTrue:[
-	    ^ self error:'invalid (unsupported) domain'.
-	].
-	addr := domainClass hostAddress:anAddressBuffer.
+        anAddressBuffer isByteArray ifFalse:[
+            ^ self error:'bad socketAddress argument'
+        ].
+        domainClass := self class socketAddressClassForDomain:domain.
+        domainClass isNil ifTrue:[
+            ^ self error:'invalid (unsupported) domain'.
+        ].
+        addr := domainClass hostAddress:anAddressBuffer.
     ].
 %{
 #ifndef NO_SOCKET
@@ -2677,86 +2711,96 @@
     if ((fp != nil)
      && __isSmallInteger(startIndex)
      && __isSmallInteger(nBytes)) {
-	SOCKET sock;
-	INT objSize;
-	struct sockaddr *sockaddr_ptr;
-	union sockaddr_u sa;
-	INT alen = 0;
-	INT sockAddrOffs, sockaddr_size;
-	INT n;
-	char *extPtr;
-	INT _flags = 0;
-	INT offs;
-	unsigned long norder;
-	unsigned char *buffer;
-	unsigned char *allocatedBuffer = NULL;
-
-	_flags = __longIntVal(flags);
-	sock = SOCKET_FROM_FILE_OBJECT(fp);
-
-	if (! __isBytes(addr)) {
-	    sockaddr_size = 0;
-	    sockaddr_ptr = (struct sockaddr *)0;
-	} else {
-	    int nIndex;
-	    OBJ cls;
-
-	    sockAddrOffs = 0;
-	    if ((cls = __qClass(addr)) != @global(ByteArray))
-		sockAddrOffs += __OBJS2BYTES__(__intVal(__ClassInstPtr(cls)->c_ninstvars));
-	    nIndex = __qSize(addr) - OHDR_SIZE;
-	    sockaddr_size = nIndex - sockAddrOffs;
-	    if (sockaddr_size > sizeof(sa)) {
-		console_fprintf(stderr, "Socket [warning]: bad socketAddr\n");
-		goto bad;
-	    }
-	    bcopy((__byteArrayVal(addr) + sockAddrOffs), &sa, sockaddr_size);
-	    sockaddr_ptr = (struct sockaddr *)(&sa);
-	}
-
-	if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
-	if (__isSmallInteger(nBytes)) {
-	    if (__intVal(nBytes) < objSize) {
-		objSize = __intVal(nBytes);
-	    }
-	}
+        SOCKET sock;
+        INT objSize;
+        struct sockaddr *sockaddr_ptr;
+        union sockaddr_u sa;
+        INT alen = 0;
+        INT sockAddrOffs, sockaddr_size;
+        INT n;
+        char *extPtr;
+        INT _flags = 0;
+        INT offs;
+        unsigned long norder;
+        unsigned char *buffer;
+        unsigned char *allocatedBuffer = NULL;
+
+        _flags = __longIntVal(flags);
+        sock = SOCKET_FROM_FILE_OBJECT(fp);
+
+        if (! __isBytes(addr)) {
+            sockaddr_size = 0;
+            sockaddr_ptr = (struct sockaddr *)0;
+        } else {
+            int nIndex;
+            OBJ cls;
+
+            sockAddrOffs = 0;
+            if ((cls = __qClass(addr)) != @global(ByteArray))
+                sockAddrOffs += __OBJS2BYTES__(__intVal(__ClassInstPtr(cls)->c_ninstvars));
+            nIndex = __qSize(addr) - OHDR_SIZE;
+            sockaddr_size = nIndex - sockAddrOffs;
+            if (sockaddr_size > sizeof(sa)) {
+                console_fprintf(stderr, "Socket [warning]: bad socketAddr\n");
+                goto bad;
+            }
+            memcpy(&sa, (__byteArrayVal(addr) + sockAddrOffs), sockaddr_size);
+            sockaddr_ptr = (struct sockaddr *)(&sa);
+        }
+
+        if (! setupBufferParameters(aDataBuffer, startIndex, &extPtr, &offs, &objSize)) goto bad;
+        if (__isSmallInteger(nBytes)) {
+            if (__intVal(nBytes) < objSize) {
+                objSize = __intVal(nBytes);
+            }
+        }
 
 #ifdef DO_WRAP_CALLS
-	if (extPtr) {
-	    buffer = extPtr + offs;
-	} else {
-	    allocatedBuffer = buffer = (char *)malloc(objSize);
-	    bcopy((char *)__InstPtr(aDataBuffer) + offs, allocatedBuffer, objSize);
-	}
-
-	do {
-	    __threadErrno = 0;
-	    n = (INT)STX_WSA_NOINT_CALL6("sendto", sendto, sock, buffer, objSize, _flags, sockaddr_ptr, sockaddr_size);
-	} while ((n < 0) && (__threadErrno == EINTR));
-
-	if (allocatedBuffer) {
-	    free(allocatedBuffer);
-	}
+        if (extPtr) {
+            buffer = extPtr + offs;
+        } else {
+            allocatedBuffer = buffer = (char *)malloc(objSize);
+            bcopy((char *)__InstPtr(aDataBuffer) + offs, allocatedBuffer, objSize);
+        }
+
+        do {
+            __threadErrno = 0;
+            n = (INT)STX_WSA_NOINT_CALL6("sendto", sendto, sock, buffer, objSize, _flags, sockaddr_ptr, sockaddr_size);
+        } while ((n < 0) && (__threadErrno == EINTR));
+        if (n < 0) {
+            errno = __threadErrno;
+        }
+
+        if (allocatedBuffer) {
+            free(allocatedBuffer);
+        }
 #else
-	__BEGIN_INTERRUPTABLE__
-	do {
-	    if (extPtr) {
-		n = sendto(sock, extPtr + offs, objSize, _flags, sockaddr_ptr, sockaddr_size);
-	    } else {
-		n = sendto(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, _flags, sockaddr_ptr, sockaddr_size);
-	    }
-	} while ((n < 0) && (errno == EINTR));
-	__END_INTERRUPTABLE__
+        __BEGIN_INTERRUPTABLE__
+        do {
+            if (extPtr) {
+                n = sendto(sock, extPtr + offs, objSize, _flags, sockaddr_ptr, sockaddr_size);
+            } else {
+                n = sendto(sock, (char *)__InstPtr(aDataBuffer) + offs, objSize, _flags, sockaddr_ptr, sockaddr_size);
+            }
+        } while ((n < 0) && (errno == EINTR));
+        __END_INTERRUPTABLE__
 #endif
 
-	if (n < 0) {
-	    __INST(lastErrorNumber) = __MKSMALLINT(errno);
-	}
-	RETURN (__MKSMALLINT(n));
+        if (n < 0) {
+            error = __INST(lastErrorNumber) = __MKSMALLINT(errno);
+        } else {
+            RETURN (__MKSMALLINT(n));
+        }            
     }
 #endif
 bad: ;
 %}.
+    error notNil ifTrue:[
+        WriteError
+            raiseRequestWith:self
+            errorString:(' : ' , OperatingSystem errorTextForNumber:error)
+    ].
+
     "
      arrive here if you try to send from an invalid buffer
      (i.e. not ByteArray-like),
@@ -2986,6 +3030,9 @@
         alen = sizeof(sa);
         newSock = (SOCKET)STX_WSA_CALL3("accept", accept, sock, &sa, &alen);
     } while ((newSock < 0) && (__threadErrno == EINTR));
+    if (ret < 0) {
+        errno = __threadErrno;
+    }
 # else
     __BEGIN_INTERRUPTABLE__
     do {
@@ -3456,9 +3503,6 @@
     addrObjSize = __qSize(addr) - nAddrInstBytes;
 
     sock = SOCKET_FROM_FILE_OBJECT(fp);
-# ifdef WIN32
-    __threadErrno = 0;
-# endif
     ret = getpeername(sock, (struct sockaddr *)&sa, &alen);
     if (ret < 0) {
 # ifdef WIN32
@@ -3533,9 +3577,6 @@
     }
 
     sock = SOCKET_FROM_FILE_OBJECT(fp);
-# ifdef WIN32
-    __threadErrno = 0;
-# endif
     ret = getsockname(sock, (struct sockaddr *)&sa, &alen);
     if (ret < 0) {
 # ifdef WIN32
@@ -4195,10 +4236,10 @@
 !Socket class methodsFor:'documentation'!
 
 version
-    ^ '$Header: /cvs/stx/stx/libbasic2/Socket.st,v 1.286 2013-10-11 17:03:01 stefan Exp $'
+    ^ '$Header: /cvs/stx/stx/libbasic2/Socket.st,v 1.287 2013-10-14 13:29:32 stefan Exp $'
 !
 
 version_CVS
-    ^ '$Header: /cvs/stx/stx/libbasic2/Socket.st,v 1.286 2013-10-11 17:03:01 stefan Exp $'
+    ^ '$Header: /cvs/stx/stx/libbasic2/Socket.st,v 1.287 2013-10-14 13:29:32 stefan Exp $'
 ! !