X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/56d8adc0c0de446235b11e9d449158d4c47b9ec8..7b9da2077d0975db6c965a85c91d5aca671ab5e3:/src/common/socket.cpp diff --git a/src/common/socket.cpp b/src/common/socket.cpp index 3a34fceba9..0f7a633165 100644 --- a/src/common/socket.cpp +++ b/src/common/socket.cpp @@ -3,14 +3,13 @@ // Purpose: Socket handler classes // Authors: Guilhem Lavaux, Guillermo Rodriguez Garcia // Created: April 1997 -// Updated: July 1999 +// Updated: September 1999 // Copyright: (C) 1999, 1998, 1997, Guilhem Lavaux +// (C) 1999, Guillermo Rodriguez Garcia // RCS_ID: $Id$ // License: see wxWindows license ///////////////////////////////////////////////////////////////////////////// -#include - #ifdef __GNUG__ #pragma implementation "socket.h" #endif @@ -45,6 +44,15 @@ #include "wx/sckaddr.h" #include "wx/socket.h" + +#if defined(__WXMSW__) || defined(__WXPM__) || defined(__WXMOTIF__) || defined(__WXMAC__) + #define PROCESS_EVENTS() wxYield() +#elif defined(__WXGTK__) + #include + #define PROCESS_EVENTS() gtk_main_iteration() +#endif + + // -------------------------------------------------------------- // ClassInfos // -------------------------------------------------------------- @@ -60,7 +68,7 @@ class wxSocketState : public wxObject public: bool notify_state; GSocketEventFlags evt_notify_state; - wxSocketBase::wxSockFlags socket_flags; + wxSockFlags socket_flags; wxSocketBase::wxSockCbk c_callback; char *c_callback_data; @@ -72,32 +80,31 @@ public: // wxSocketBase ctor and dtor // -------------------------------------------------------------- -wxSocketBase::wxSocketBase(wxSocketBase::wxSockFlags _flags, - wxSocketBase::wxSockType _type) : +wxSocketBase::wxSocketBase(wxSockFlags _flags, wxSockType _type) : wxEvtHandler(), - m_socket(NULL), m_flags(_flags), m_type(_type), - m_neededreq(0), - m_lcount(0), m_timeout(600), - m_unread(NULL), m_unrd_size(0), m_unrd_cur(0), - m_cbk(NULL), m_cdata(NULL), + m_socket(NULL), m_id(-1), + m_flags(_flags), m_type(_type), + m_neededreq(0), m_notify_state(FALSE), m_connected(FALSE), m_establishing(FALSE), - m_notify_state(FALSE), m_id(-1), - m_defering(NO_DEFER), - m_states() + m_reading(FALSE), m_writing(FALSE), + m_error(FALSE), m_lcount(0), m_timeout(600), m_states(), + m_unread(NULL), m_unrd_size(0), m_unrd_cur(0), + m_defering(NO_DEFER), m_defer_buffer(NULL), m_defer_timer(NULL), + m_cbk(NULL), m_cdata(NULL) { } wxSocketBase::wxSocketBase() : wxEvtHandler(), - m_socket(NULL), m_flags(WAITALL | SPEED), m_type(SOCK_UNINIT), - m_neededreq(0), - m_lcount(0), m_timeout(600), - m_unread(NULL), m_unrd_size(0), m_unrd_cur(0), - m_cbk(NULL), m_cdata(NULL), + m_socket(NULL), m_id(-1), + m_flags(NONE), m_type(SOCK_UNINIT), + m_neededreq(0), m_notify_state(FALSE), m_connected(FALSE), m_establishing(FALSE), - m_notify_state(FALSE), m_id(-1), - m_defering(NO_DEFER), - m_states() + m_reading(FALSE), m_writing(FALSE), + m_error(FALSE), m_lcount(0), m_timeout(600), m_states(), + m_unread(NULL), m_unrd_size(0), m_unrd_cur(0), + m_defering(NO_DEFER), m_defer_buffer(NULL), m_defer_timer(NULL), + m_cbk(NULL), m_cdata(NULL) { } @@ -135,13 +142,10 @@ bool wxSocketBase::Close() // wxSocketBase basic IO operations // -------------------------------------------------------------- -// GRG: I have made some changes to wxSocket internal event -// system; now, all events (INPUT, OUTPUT, CONNECTION, LOST) -// are always internally monitored; but users will only be -// notified of these events they are interested in. So we -// no longer have to change the event mask with SetNotify() -// in internal functions like DeferRead, DeferWrite, and -// the like. This solves a lot of problems. +// All IO operations {Read, Write, ReadMsg, WriteMsg, Peek, +// Unread, Discard} update m_error and m_lcount. +// +// TODO: Should Connect, Accept and AcceptWith update m_error? class _wxSocketInternalTimer: public wxTimer { @@ -155,11 +159,12 @@ public: } }; -int wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes) +wxUint32 wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes) { // Timer for timeout _wxSocketInternalTimer timer; + //wxLogMessage("Entrando a DeferRead, nbytes = %d", nbytes); wxASSERT(m_defering == NO_DEFER); // Set the defering mode to READ. @@ -177,11 +182,14 @@ int wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes) // If the socket is readable, call DoDefer for the first time if (GSocket_Select(m_socket, GSOCK_INPUT_FLAG)) + { + //wxLogMessage("Llamando al primer DoDefer"); DoDefer(); + } - // Wait for buffer completion. + // Wait for buffer completion. while (m_defer_buffer != NULL) - wxYield(); + PROCESS_EVENTS(); timer.Stop(); @@ -190,129 +198,228 @@ int wxSocketBase::DeferRead(char *buffer, wxUint32 nbytes) m_defer_timer = NULL; // Return the number of bytes read from the socket. + //wxLogMessage("Saliendo de DeferRead: total: %d bytes", nbytes - m_defer_nbytes); return nbytes-m_defer_nbytes; } wxSocketBase& wxSocketBase::Read(char* buffer, wxUint32 nbytes) { - int ret = 1; + // Mask read events + m_reading = TRUE; - m_lcount = GetPushback(buffer, nbytes, FALSE); - nbytes -= m_lcount; - buffer += m_lcount; + m_lcount = _Read(buffer, nbytes); - if (!m_connected) - return *this; + // If in WAITALL mode, all bytes should have been read. + if (m_flags & WAITALL) + m_error = (m_lcount != nbytes); + else + m_error = (m_lcount == 0); + + // Trigger another read event if there is still data available. + m_reading = FALSE; + // TODO: TriggerRead + + return *this; +} - // If we have got the whole needed buffer, return immediately - if (!nbytes) +wxUint32 wxSocketBase::_Read(char* buffer, wxUint32 nbytes) +{ + int total; + int ret = 1; + + // we try this even if the connection has already been closed. + total = GetPushback(buffer, nbytes, FALSE); + nbytes -= total; + buffer += total; + + // If the socket is not connected, or we have got the whole + // needed buffer, return immedately + if (!m_connected || !m_socket || !nbytes) + return total; + + // Possible combinations (they are checked in this order) + // NOWAIT + // SPEED | WAITALL + // SPEED + // WAITALL + // NONE + // + if (m_flags & NOWAIT) // NOWAIT { - return *this; - } + GSocket_SetNonBlocking(m_socket, TRUE); + ret = GSocket_Read(m_socket, buffer, nbytes); + GSocket_SetNonBlocking(m_socket, FALSE); - if (m_flags & SPEED & WAITALL) // SPEED && WAITALL + if (ret > 0) + total += ret; + } + else if ((m_flags & SPEED) && (m_flags & WAITALL)) // SPEED, WAITALL { while (ret > 0 && nbytes > 0) { ret = GSocket_Read(m_socket, buffer, nbytes); - m_lcount += ret; + total += ret; buffer += ret; nbytes -= ret; } // In case the last call was an error ... if (ret < 0) - m_lcount ++; + total++; } - else if (m_flags & SPEED) // SPEED && !WAITALL + else if (m_flags & SPEED) // SPEED, !WAITALL { ret = GSocket_Read(m_socket, buffer, nbytes); if (ret > 0) - m_lcount += ret; + total += ret; } - else // !SPEED + else // NONE or WAITALL { ret = DeferRead(buffer, nbytes); if (ret > 0) - m_lcount += ret; + total += ret; } - return *this; + return total; } wxSocketBase& wxSocketBase::ReadMsg(char* buffer, wxUint32 nbytes) { - unsigned long len, len2, sig; - struct { - char sig[4]; - char len[4]; +#define MAX_DISCARD_SIZE (10 * 1024) + + wxUint32 len, len2, sig, total; + bool error; + int old_flags; + struct + { + unsigned char sig[4]; + unsigned char len[4]; } msg; - // sig should be an explicit 32-bit unsigned integer; I've seen - // compilers in which wxUint32 was actually a 16-bit unsigned integer + // Mask read events + m_reading = TRUE; + + total = 0; + error = TRUE; + old_flags = m_flags; + SetFlags((m_flags & SPEED) | WAITALL); - Read((char *)&msg, sizeof(msg)); - if (m_lcount != sizeof(msg)) - return *this; + if (_Read((char *)&msg, sizeof(msg)) != sizeof(msg)) + goto exit; - sig = msg.sig[0] & 0xff; - sig |= (wxUint32)(msg.sig[1] & 0xff) << 8; - sig |= (wxUint32)(msg.sig[2] & 0xff) << 16; - sig |= (wxUint32)(msg.sig[3] & 0xff) << 24; + sig = (wxUint32)msg.sig[0]; + sig |= (wxUint32)(msg.sig[1] << 8); + sig |= (wxUint32)(msg.sig[2] << 16); + sig |= (wxUint32)(msg.sig[3] << 24); if (sig != 0xfeeddead) - return *this; - len = msg.len[0] & 0xff; - len |= (wxUint32)(msg.len[1] & 0xff) << 8; - len |= (wxUint32)(msg.len[2] & 0xff) << 16; - len |= (wxUint32)(msg.len[3] & 0xff) << 24; - - // len2 is incorrectly computed in the original; this sequence is - // the fix - if (len > nbytes) { + { + wxLogMessage(wxT("Warning: invalid signature returned to ReadMsg")); + goto exit; + } + + len = (wxUint32)msg.len[0]; + len |= (wxUint32)(msg.len[1] << 8); + len |= (wxUint32)(msg.len[2] << 16); + len |= (wxUint32)(msg.len[3] << 24); + + //wxLogMessage("Readmsg: %d %d %d %d -> len == %d", + // msg.len[0], msg.len[1], msg.len[2], msg.len[3], len); + + if (len > nbytes) + { len2 = len - nbytes; len = nbytes; } else len2 = 0; - // the "len &&" in the following statement is necessary so that - // we don't attempt to read (and possibly hang the system) - // if the message was zero bytes long - if (len && Read(buffer, len).LastCount() != len) - return *this; - if (len2 && (Read(NULL, len2).LastCount() != len2)) - return *this; - if (Read((char *)&msg, sizeof(msg)).LastCount() != sizeof(msg)) - return *this; - - sig = msg.sig[0] & 0xff; - sig |= (wxUint32)(msg.sig[1] & 0xff) << 8; - sig |= (wxUint32)(msg.sig[2] & 0xff) << 16; - sig |= (wxUint32)(msg.sig[3] & 0xff) << 24; - -// ERROR + // This check is necessary so that we don't attemp to read if + // the msg was zero bytes long. + if (len) + { + total = _Read(buffer, len); + + if (total != len) + goto exit; + } + if (len2) + { + char *discard_buffer = new char[MAX_DISCARD_SIZE]; + long discard_len; + + // NOTE: discarded bytes don't add to m_lcount. + do + { + discard_len = ((len2 > MAX_DISCARD_SIZE)? MAX_DISCARD_SIZE : len2); + discard_len = _Read(discard_buffer, discard_len); + len2 -= discard_len; + } + while ((discard_len > 0) && len2); + + delete [] discard_buffer; + + if (len2 != 0) + goto exit; + } + if (_Read((char *)&msg, sizeof(msg)) != sizeof(msg)) + goto exit; + + sig = (wxUint32)msg.sig[0]; + sig |= (wxUint32)(msg.sig[1] << 8); + sig |= (wxUint32)(msg.sig[2] << 16); + sig |= (wxUint32)(msg.sig[3] << 24); + if (sig != 0xdeadfeed) - wxLogDebug(_T("Warning: invalid signature returned to ReadMsg\n")); + { + //wxLogMessage(wxT("Warning: invalid signature returned to ReadMsg")); + goto exit; + } + // everything was OK + error = FALSE; + +exit: + m_error = error; + m_lcount = total; + m_reading = FALSE; + SetFlags(old_flags); + + // TODO: TriggerRead return *this; + +#undef MAX_DISCARD_SIZE } wxSocketBase& wxSocketBase::Peek(char* buffer, wxUint32 nbytes) { - Read(buffer, nbytes); - CreatePushbackAfter(buffer, nbytes); + // Mask read events + m_reading = TRUE; + + m_lcount = _Read(buffer, nbytes); + Pushback(buffer, nbytes); + + // If in WAITALL mode, all bytes should have been read. + if (m_flags & WAITALL) + m_error = (m_lcount != nbytes); + else + m_error = (m_lcount == 0); + + // Trigger another read event if there is still data available. + m_reading = FALSE; + // TODO: TriggerRead return *this; } -int wxSocketBase::DeferWrite(const char *buffer, wxUint32 nbytes) +wxUint32 wxSocketBase::DeferWrite(const char *buffer, wxUint32 nbytes) { // Timer for timeout _wxSocketInternalTimer timer; wxASSERT(m_defering == NO_DEFER); + //wxLogMessage("Entrando a DeferWrite"); m_defering = DEFER_WRITE; @@ -329,125 +436,188 @@ int wxSocketBase::DeferWrite(const char *buffer, wxUint32 nbytes) // If the socket is writable, call DoDefer for the first time if (GSocket_Select(m_socket, GSOCK_OUTPUT_FLAG)) + { + //wxLogMessage("Llamando al primer DoDefer"); DoDefer(); + } - // Wait for buffer completion. + // Wait for buffer completion. while (m_defer_buffer != NULL) - wxYield(); + PROCESS_EVENTS(); - // Stop timer - m_defer_timer = NULL; timer.Stop(); + // Disable defering mode + m_defer_timer = NULL; m_defering = NO_DEFER; + wxString s; + s.Printf(wxT("Saliendo de DeferWrite: total %d bytes"), nbytes-m_defer_nbytes); + //wxLogMessage(s); + return nbytes-m_defer_nbytes; } wxSocketBase& wxSocketBase::Write(const char *buffer, wxUint32 nbytes) { + // Mask write events + m_writing = TRUE; + + m_lcount = _Write(buffer, nbytes); + + // If in WAITALL mode, all bytes should have been written. + if (m_flags & WAITALL) + m_error = (m_lcount != nbytes); + else + m_error = (m_lcount == 0); + + // Trigger another write event if the socket is still writable + m_writing = FALSE; + + // TODO: TriggerWrite + return *this; +} + +wxUint32 wxSocketBase::_Write(const char *buffer, wxUint32 nbytes) +{ + wxUint32 total = 0; int ret = 1; - m_lcount = 0; + if (!m_connected || !m_socket) + return 0; - if (!m_connected) - return *this; + // Possible combinations (they are checked in this order) + // NOWAIT + // SPEED | WAITALL + // SPEED + // WAITALL + // NONE + // + if (m_flags & NOWAIT) // NOWAIT + { + GSocket_SetNonBlocking(m_socket, TRUE); + ret = GSocket_Write(m_socket, buffer, nbytes); + GSocket_SetNonBlocking(m_socket, FALSE); - if (m_flags & SPEED & WAITALL) // SPEED && WAITALL + if (ret > 0) + total = ret; + } + else if ((m_flags & SPEED) && (m_flags & WAITALL)) // SPEED, WAITALL { while (ret > 0 && nbytes > 0) { ret = GSocket_Write(m_socket, buffer, nbytes); - m_lcount += ret; + total += ret; buffer += ret; nbytes -= ret; } // In case the last call was an error ... if (ret < 0) - m_lcount ++; + total ++; } - else if (m_flags & SPEED) // SPEED && !WAITALL + else if (m_flags & SPEED) // SPEED, !WAITALL { ret = GSocket_Write(m_socket, buffer, nbytes); if (ret > 0) - m_lcount += ret; + total = ret; } - else // !SPEED + else // NONE or WAITALL { ret = DeferWrite(buffer, nbytes); if (ret > 0) - m_lcount += ret; + total = ret; } - return *this; + return total; } wxSocketBase& wxSocketBase::WriteMsg(const char *buffer, wxUint32 nbytes) { + wxUint32 total; + bool error; + int old_flags; struct { - char sig[4]; - char len[4]; + unsigned char sig[4]; + unsigned char len[4]; } msg; + // Mask write events + m_writing = TRUE; + + error = TRUE; + total = 0; + old_flags = m_flags; + SetFlags((m_flags & SPEED) | WAITALL); + // warning about 'cast truncates constant value' #ifdef __VISUALC__ - #pragma warning(disable: 4310) +# pragma warning(disable: 4310) #endif // __VISUALC__ - msg.sig[0] = (char) 0xad; - msg.sig[1] = (char) 0xde; - msg.sig[2] = (char) 0xed; - msg.sig[3] = (char) 0xfe; - - msg.len[0] = (char) nbytes & 0xff; - msg.len[1] = (char) (nbytes >> 8) & 0xff; - msg.len[2] = (char) (nbytes >> 16) & 0xff; - msg.len[3] = (char) (nbytes >> 24) & 0xff; - - if (Write((char *)&msg, sizeof(msg)).LastCount() < sizeof(msg)) - return *this; - if (Write(buffer, nbytes).LastCount() < nbytes) - return *this; - - msg.sig[0] = (char) 0xed; - msg.sig[1] = (char) 0xfe; - msg.sig[2] = (char) 0xad; - msg.sig[3] = (char) 0xde; + msg.sig[0] = (unsigned char) 0xad; + msg.sig[1] = (unsigned char) 0xde; + msg.sig[2] = (unsigned char) 0xed; + msg.sig[3] = (unsigned char) 0xfe; + + msg.len[0] = (unsigned char) nbytes & 0xff; + msg.len[1] = (unsigned char) (nbytes >> 8) & 0xff; + msg.len[2] = (unsigned char) (nbytes >> 16) & 0xff; + msg.len[3] = (unsigned char) (nbytes >> 24) & 0xff; + + //wxLogMessage("Writemsg: %d %d %d %d -> %d", + // nbytes & 0xff, + // (nbytes >> 8) & 0xff, + // (nbytes >> 16) & 0xff, + // (nbytes >> 24) & 0xff, + // nbytes + // ); + + if (_Write((char *)&msg, sizeof(msg)) < sizeof(msg)) + goto exit; + + total = _Write(buffer, nbytes); + + if (total < nbytes) + goto exit; + + msg.sig[0] = (unsigned char) 0xed; + msg.sig[1] = (unsigned char) 0xfe; + msg.sig[2] = (unsigned char) 0xad; + msg.sig[3] = (unsigned char) 0xde; msg.len[0] = msg.len[1] = msg.len[2] = msg.len[3] = (char) 0; - Write((char *)&msg, sizeof(msg)); + if ((_Write((char *)&msg, sizeof(msg))) < sizeof(msg)) + goto exit; + + // everything was OK + error = FALSE; + +exit: + m_error = error; + m_lcount = total; + m_writing = FALSE; + + // TODO: TriggerWrite return *this; #ifdef __VISUALC__ - #pragma warning(default: 4310) +# pragma warning(default: 4310) #endif // __VISUALC__ } wxSocketBase& wxSocketBase::Unread(const char *buffer, wxUint32 nbytes) { - m_lcount = 0; if (nbytes != 0) - { - CreatePushbackAfter(buffer, nbytes); - m_lcount = nbytes; - } - return *this; -} + Pushback(buffer, nbytes); -bool wxSocketBase::IsData() const -{ - if (!m_socket) - return FALSE; + m_error = FALSE; + m_lcount = nbytes; - return (GSocket_Select(m_socket, GSOCK_INPUT_FLAG)); + return *this; } -// GRG: DoDefer() no longer needs to know which event occured, -// because this was only used to catch LOST events and set -// m_defer_buffer = NULL; this is done in OnRequest() now. - void wxSocketBase::DoDefer() { int ret; @@ -455,7 +625,7 @@ void wxSocketBase::DoDefer() if (!m_defer_buffer) return; - switch (m_defering) + switch(m_defering) { case DEFER_READ: ret = GSocket_Read(m_socket, m_defer_buffer, m_defer_nbytes); @@ -465,7 +635,6 @@ void wxSocketBase::DoDefer() break; default: ret = -1; - break; } if (ret >= 0) @@ -473,7 +642,7 @@ void wxSocketBase::DoDefer() // If we are waiting for all bytes to be acquired, keep the defering // mode enabled. - if ((m_flags & WAITALL) == 0 || m_defer_nbytes == 0 || ret < 0) + if (!(m_flags & WAITALL) || !m_defer_nbytes || ret < 0) { m_defer_buffer = NULL; } @@ -482,25 +651,40 @@ void wxSocketBase::DoDefer() m_defer_buffer += ret; m_defer_timer->Start(m_timeout * 1000, FALSE); } + + //wxLogMessage("DoDefer ha transferido %d bytes", ret); } -void wxSocketBase::Discard() +wxSocketBase& wxSocketBase::Discard() { #define MAX_BUFSIZE (10*1024) + int old_flags; char *my_data = new char[MAX_BUFSIZE]; wxUint32 recv_size = MAX_BUFSIZE; + wxUint32 total = 0; + + // Mask read events + m_reading = TRUE; - SaveState(); - SetFlags(NOWAIT | SPEED); + old_flags = m_flags; + SetFlags(NOWAIT); while (recv_size == MAX_BUFSIZE) { - recv_size = Read(my_data, MAX_BUFSIZE).LastCount(); + recv_size = _Read(my_data, MAX_BUFSIZE); + total += recv_size; } - RestoreState(); delete [] my_data; + m_lcount = total; + m_error = FALSE; + + // Trigger another read event if there is still data available. + m_reading = FALSE; + + // TODO: TriggerRead + return *this; #undef MAX_BUFSIZE } @@ -587,7 +771,7 @@ void wxSocketBase::RestoreState() // they poll the socket, using GSocket_Select(), to check for // the specified combination of event flags, until an event // occurs or until the timeout ellapses. The polling loop -// calls wxYield(), so this won't block the GUI. +// calls PROCESS_EVENTS(), so this won't block the GUI. bool wxSocketBase::_Wait(long seconds, long milliseconds, wxSocketEventFlags flags) { @@ -597,7 +781,11 @@ bool wxSocketBase::_Wait(long seconds, long milliseconds, wxSocketEventFlags fla int state = -1; // Check for valid socket - if ((!m_connected && !m_establishing) || !m_socket) + if (!m_socket) + return FALSE; + + // If it is not a server, it must be connected or establishing connection + if ((m_type != SOCK_SERVER) && (!m_connected && !m_establishing)) return FALSE; // Check for valid timeout value @@ -607,22 +795,66 @@ bool wxSocketBase::_Wait(long seconds, long milliseconds, wxSocketEventFlags fla timeout = m_timeout * 1000; // Activate timer - timer.m_state = &state; - timer.m_new_val = 0; - timer.Start(timeout, TRUE); + if (timeout) + { + timer.m_state = &state; + timer.m_new_val = 0; + timer.Start(timeout, TRUE); + } // Active polling (without using events) - result = GSocket_Select(m_socket, flags); - - while ((result == 0) && (state == -1)) + // + // NOTE: this duplicates some of the code in OnRequest (lost + // connection and connection establishment handling) but this + // doesn't hurt. It has to be here because the event might + // be a bit delayed, and it has to be in OnRequest as well + // because maybe the WaitXXX functions are not being used. + // + // Do this at least once (important if timeout == 0, when + // we are just polling) + do { - wxYield(); - result = GSocket_Select(m_socket, flags); + result = GSocket_Select(m_socket, flags | GSOCK_LOST_FLAG); + + // Connection lost + if (result & GSOCK_LOST_FLAG) + { + timer.Stop(); + m_defer_buffer = NULL; + Close(); + return TRUE; + } + + // Incoming connection (server) or connection established (client) + if (result & GSOCK_CONNECTION_FLAG) + { + timer.Stop(); + m_connected = TRUE; + m_establishing = FALSE; + return TRUE; + } + + // If we are in the middle of a R/W operation, do not propagate + // to users. + if ( ((result & GSOCK_INPUT_FLAG) || (result & GSOCK_OUTPUT_FLAG)) + && (!m_writing) && (!m_reading) ) + { +/* TODO: remove this + if (m_defer_buffer == NULL) +*/ + { + timer.Stop(); + return TRUE; + } + } + + if (timeout != 0) + PROCESS_EVENTS(); } + while ((state == -1) && timeout); timer.Stop(); - - return (result != 0); + return FALSE; } bool wxSocketBase::Wait(long seconds, long milliseconds) @@ -635,12 +867,16 @@ bool wxSocketBase::Wait(long seconds, long milliseconds) bool wxSocketBase::WaitForRead(long seconds, long milliseconds) { - return _Wait(seconds, milliseconds, GSOCK_INPUT_FLAG | GSOCK_LOST_FLAG); + // Check pushback buffer + if (m_unread) + return TRUE; + + return _Wait(seconds, milliseconds, GSOCK_INPUT_FLAG); } bool wxSocketBase::WaitForWrite(long seconds, long milliseconds) { - return _Wait(seconds, milliseconds, GSOCK_OUTPUT_FLAG | GSOCK_LOST_FLAG); + return _Wait(seconds, milliseconds, GSOCK_OUTPUT_FLAG); } bool wxSocketBase::WaitForLost(long seconds, long milliseconds) @@ -648,6 +884,14 @@ bool wxSocketBase::WaitForLost(long seconds, long milliseconds) return _Wait(seconds, milliseconds, GSOCK_LOST_FLAG); } +void wxSocketBase::SetTimeout(long seconds) +{ + m_timeout = seconds; + + if (m_socket) + GSocket_SetTimeout(m_socket, m_timeout); +} + // -------------------------------------------------------------- // wxSocketBase flags // -------------------------------------------------------------- @@ -657,11 +901,6 @@ void wxSocketBase::SetFlags(wxSockFlags _flags) m_flags = _flags; } -wxSocketBase::wxSockFlags wxSocketBase::GetFlags() const -{ - return m_flags; -} - // -------------------------------------------------------------- // wxSocketBase callback management // -------------------------------------------------------------- @@ -686,7 +925,12 @@ char *wxSocketBase::CallbackData(char *data) // wxSocketBase automatic notifier // -------------------------------------------------------------- -static void wx_socket_callback(GSocket *socket, GSocketEvent event, char *cdata) +// All events (INPUT, OUTPUT, CONNECTION, LOST) are now always +// internally watched; but users will only be notified of those +// events they are interested in. + +static void LINKAGEMODE wx_socket_callback(GSocket * WXUNUSED(socket), + GSocketEvent event, char *cdata) { wxSocketBase *sckobj = (wxSocketBase *)cdata; @@ -720,6 +964,14 @@ void wxSocketBase::OnRequest(wxSocketNotify req_evt) wxSocketEvent event(m_id); wxSocketEventFlags flag = EventToNotify(req_evt); + // fprintf(stderr, "%s: Entering OnRequest (evt %d)\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt); + + // NOTE: this duplicates some of the code in _Wait (lost + // connection and connection establishment handling) but + // this doesn't hurt. It has to be here because maybe the + // WaitXXX are not being used, and it has to be in _Wait + // as well because the event might be a bit delayed. + // switch(req_evt) { case wxSOCKET_CONNECTION: @@ -734,10 +986,7 @@ void wxSocketBase::OnRequest(wxSocketNotify req_evt) case wxSOCKET_OUTPUT: if (m_defer_buffer) { - // GRG: DoDefer() no longer needs to know which - // event occured, because this was only used to - // catch LOST events and set m_defer_buffer to - // NULL, and this is done in OnRequest() now. + // fprintf(stderr, "%s: Habia buffer, evt %d skipped\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt); DoDefer(); // Do not notify to user return; @@ -745,8 +994,18 @@ void wxSocketBase::OnRequest(wxSocketNotify req_evt) break; } + // If we are in the middle of a R/W operation, do not + // propagate events to users. + if (((req_evt == wxSOCKET_INPUT) && m_reading) || + ((req_evt == wxSOCKET_OUTPUT) && m_writing)) + { + // fprintf(stderr, "%s: Swallowed evt %d\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt); + return; + } + if (((m_neededreq & flag) == flag) && m_notify_state) { + // fprintf(stderr, "%s: Evt %d delivered\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt); event.m_socket = this; event.m_skevt = req_evt; ProcessEvent(event); @@ -755,9 +1014,11 @@ void wxSocketBase::OnRequest(wxSocketNotify req_evt) if (m_cbk) m_cbk(*this, req_evt, m_cdata); } + + // fprintf(stderr, "%s: Exiting OnRequest (evt %d)\n", (m_type == SOCK_CLIENT)? "client" : "server", req_evt); } -void wxSocketBase::OldOnNotify(wxSocketNotify evt) +void wxSocketBase::OldOnNotify(wxSocketNotify WXUNUSED(evt)) { } @@ -772,25 +1033,10 @@ void wxSocketBase::SetEventHandler(wxEvtHandler& h_evt, int id) } // -------------------------------------------------------------- -// wxSocketBase pushback library +// wxSocketBase pushback // -------------------------------------------------------------- -void wxSocketBase::CreatePushbackAfter(const char *buffer, wxUint32 size) -{ - char *curr_pos; - - if (m_unread != NULL) - m_unread = (char *) realloc(m_unread, m_unrd_size+size); - else - m_unread = (char *) malloc(size); - - curr_pos = m_unread + m_unrd_size; - - memcpy(curr_pos, buffer, size); - m_unrd_size += size; -} - -void wxSocketBase::CreatePushbackBefore(const char *buffer, wxUint32 size) +void wxSocketBase::Pushback(const char *buffer, wxUint32 size) { if (m_unread == NULL) m_unread = (char *)malloc(size); @@ -816,6 +1062,7 @@ wxUint32 wxSocketBase::GetPushback(char *buffer, wxUint32 size, bool peek) if (size > (m_unrd_size-m_unrd_cur)) size = m_unrd_size-m_unrd_cur; + memcpy(buffer, (m_unread+m_unrd_cur), size); if (!peek) { @@ -873,6 +1120,9 @@ bool wxSocketServer::AcceptWith(wxSocketBase& sock, bool wait) { GSocket *child_socket; + if (!m_socket) + return FALSE; + // GRG: If wait == FALSE, then the call should be nonblocking. // When we are finished, we put the socket to blocking mode // again. @@ -913,9 +1163,9 @@ wxSocketBase *wxSocketServer::Accept(bool wait) return sock; } -bool wxSocketServer::WaitOnAccept(long seconds, long milliseconds) +bool wxSocketServer::WaitForAccept(long seconds, long milliseconds) { - return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG | GSOCK_LOST_FLAG); + return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG); } // -------------------------------------------------------------- @@ -958,6 +1208,9 @@ bool wxSocketClient::Connect(wxSockAddress& addr_man, bool wait) return FALSE; GSocket_SetTimeout(m_socket, m_timeout); + GSocket_SetCallback(m_socket, GSOCK_INPUT_FLAG | GSOCK_OUTPUT_FLAG | + GSOCK_LOST_FLAG | GSOCK_CONNECTION_FLAG, + wx_socket_callback, (char *)this); // GRG: If wait == FALSE, then the call should be nonblocking. // When we are finished, we put the socket to blocking mode @@ -968,9 +1221,6 @@ bool wxSocketClient::Connect(wxSockAddress& addr_man, bool wait) GSocket_SetPeer(m_socket, addr_man.GetAddress()); err = GSocket_Connect(m_socket, GSOCK_STREAMED); - GSocket_SetCallback(m_socket, GSOCK_INPUT_FLAG | GSOCK_OUTPUT_FLAG | - GSOCK_LOST_FLAG | GSOCK_CONNECTION_FLAG, - wx_socket_callback, (char *)this); if (!wait) GSocket_SetNonBlocking(m_socket, FALSE); @@ -989,39 +1239,15 @@ bool wxSocketClient::Connect(wxSockAddress& addr_man, bool wait) bool wxSocketClient::WaitOnConnect(long seconds, long milliseconds) { - bool ret; - - if (m_connected) // Already connected + if (m_connected) // Already connected return TRUE; - if (!m_establishing) // No connection in progress + if (!m_establishing || !m_socket) // No connection in progress return FALSE; - ret = _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG | GSOCK_LOST_FLAG); - - // GRG: m_connected and m_establishing will be updated in - // OnRequest(), but we do it here anyway because sometimes - // the event might be a bit delayed, and if that happens, - // when WaitOnConnect() returns, m_connected will still be - // FALSE. We have to do it as well in OnRequest because - // maybe WaitOnConnect() is not being used... - - if (ret) - { - m_connected = GSocket_Select(m_socket, GSOCK_CONNECTION_FLAG); - m_establishing = FALSE; - } - - return m_connected; + return _Wait(seconds, milliseconds, GSOCK_CONNECTION_FLAG); } - -void wxSocketClient::OnRequest(wxSocketNotify req_evt) -{ - wxSocketBase::OnRequest(req_evt); -} - - // -------------------------------------------------------------- // wxSocketEvent // --------------------------------------------------------------