Die unten stehende Funktion macht mir einige Problemchen. Wenn die Funktion mit 0 zurückkehr wird ein neuer Thread gestartet der den neuen Client mit accept annimmt und komplett im Thread bearbeitet.
Leider ist das Problem, das diese Funktion an der letzten!!! Klammer mit einer Access violation (reading) 0xfdfdfdfd verläßt. Ohne Debugger schmiert der Server ohne einen Mucks ab, weder Fehlermeldung noch sonstiges!
Release und Debugger bleibt er irgendwo in einer Mutexsache (lock / unlock) stecken.

Hat hierfür irgendeine Lösung oder erklärung?

Betreffende Funktion:

int socket_select(SOCKINFO &sockinfo)
	{
		if(sockinfo.socket != INVALID_SOCKET)
		{
			FD_SET fd_read;
			FD_ZERO(&fd_read);
			FD_SET(sockinfo.socket, &fd_read);

			if(SOCKET_ERROR == select(NULL, &fd_read, NULL, NULL, NULL))
			{
				return WSAGetLastError();
			}

			if(FD_ISSET(sockinfo.socket, &fd_read))
			{
				return 0;
			}
			return 1;
		}
		return 1;
	} // Hier schmiert er im Debugmodus im Debugger ab.

Der Aufruf:

while(true)
			{
				if(!socket_select(server))
				{
					CreateThread(NULL, NULL, ClientProc, this, NULL, NULL);
				}
			}

if(SOCKET_ERROR == select(NULL, &fd_read, NULL, NULL, NULL))
{
  return WSAGetLastError();
}

immer aus. Das liegt daran, dass select als 'potentiell blockierend' eingestuft wird. Du solltest also vorher das Flag WSAWOULDBLOCK ausschließen. Könnte sein, dass es dadurch zu Folgefehlern kommt, weiß aber nicht, ob das Dein Problem lößt.

Wie gesagt ist das phänomen unerklärlich, über das lokale netzwerk kackt er aber wesentlich seltener ab als mit localhost. (???)

typedef struct _SOCKINFO
	{
		SOCKET socket;
		SOCKADDR_IN addr;
		int addr_size;
	}SOCKINFO;

int socket_init()
	{
		WSAData wsa;
		if(WSAStartup(MAKEWORD(2, 0), &wsa) != 0)
			return 1;
		else
			return 0;
	}

int socket_listen(SOCKINFO & sockinfo, const int port)
	{
		sockinfo.socket = INVALID_SOCKET;

		if(INVALID_SOCKET == (sockinfo.socket = socket(AF_INET, SOCK_STREAM, 0)))
		{
			return WSAGetLastError();
		}

		memset(&sockinfo.addr, 0, sizeof(sockinfo.addr));
		sockinfo.addr.sin_family = AF_INET;
		sockinfo.addr.sin_port = htons(port);

		if(SOCKET_ERROR == bind(sockinfo.socket, (SOCKADDR*)&sockinfo.addr, sizeof(sockinfo.addr)))
		{
			return WSAGetLastError();
		}

		if(SOCKET_ERROR == listen(sockinfo.socket, 20))
		{
			return WSAGetLastError();
		}

		return 0;
	}

int socket_accept(SOCKINFO &sockinfo, SOCKINFO &new_sockinfo)
	{
		if(sockinfo.socket != INVALID_SOCKET)
		{
			new_sockinfo.socket = INVALID_SOCKET;
			new_sockinfo.addr_size = sizeof(new_sockinfo.addr);

			if(INVALID_SOCKET == (new_sockinfo.socket = accept(sockinfo.socket, (SOCKADDR*)&new_sockinfo.addr, &new_sockinfo.addr_size)))
			{
				return WSAGetLastError();
			}
			return 0;
		}
		return 1;
	}

socket_select(...) läuft in einer quasi endlosschleife (blockierend), sobald es zuckt wird socket_accept aufgerufen (in einem Thread)

bisher find ich siehst nämlich ganz OK aus

Die Schleife sieht man oben schon mal, mehr ist das nicht.

Hab Debugausgaben, nur beim Server kann man sie nicht mehr lesen so schnell ist er weg, bzw hängt es nicht an der Funktion, sondern an etwas anderem wie man auf dem Bild sehen kann. Hab im DOS Fenster ausgaben zum WSAGetLastError() und MessagesBoxen für da wo er grad ist, die kommen aber nie, also kackt er vorher ab.

Wenn die Funktion mit 0 zurückkehr wird ein neuer Thread gestartet der den neuen Client mit accept annimmt und komplett im Thread bearbeitet.

Wieviele Threads werden im Debug-Fenster unten links angezeigt in dem Augenblick wo die Exception ausgelöst wird ?

3520	__tmainCRTStartup	   std::_Lockit::~_Lockit	Normal	0
>	2476	__tmainCRTStartup	   _unlock	                  Normal	0

2 stück

Das bedeutet ja, dass ein Client "angebissen" hat, der Server ein Thread erzeugt und daraufhin abstürzt.

Kannst Du mal die "ClientProc" von "CreateThread(NULL, NULL, ClientProc, this, NULL, NULL)" posten ?

int SmartSockServer::Run()
	{
		SOCKINFO client;
		char buffer[256];
		char text[256];
		int res;
		int bytes;
		int number;

		if((res = socket_accept(server, client)))
			return res;
		else
		{
			LeaveCriticalSection(&crit);

			socket_recv(client, buffer, 256, bytes);
			buffer[bytes] = '\0';
			sscanf_s(buffer, "%i %s", &number, text);
			if(number == 100)
			{
				if(strstr(text, "admin") == 0)
				{
					MessageBox(NULL, "USER OK", "Debug", NULL);
					sprintf_s(buffer, 256, "%i", 110);
					socket_send(client, buffer, 256, (int)strlen(buffer));
					socket_recv(client, buffer, 256, bytes);
					buffer[bytes] = '\0';
					sscanf_s(buffer, "%i %s", &number, text);
					if(number == 130)
					{
						if(strstr(text, "admin") == 0)
						{
							sprintf_s(buffer, 256, "%i", 140);
							socket_send(client, buffer, 256, (int)strlen(buffer));
							MessageBox(NULL, "PASS OK", "Debug", NULL);
						}
						else
						{
							sprintf_s(buffer, 256, "%i", 150);
							socket_send(client, buffer, 256, (int)strlen(buffer));
							socket_close(client);
							MessageBox(NULL, text, "PASS NOT OK", NULL);
							return 0;
						}
					}
				}
				else
				{
					sprintf_s(buffer, 256, "%i", 120);
					socket_send(client, buffer, 256, (int)strlen(buffer));
					socket_close(client);
					MessageBox(NULL, text, "USER NOT OK", NULL);
					return 0;
				}
			}
		}
		MessageBox(NULL, "We leave te thread", "Debug", NULL);
		return 0;
	}

	DWORD WINAPI ClientProc(LPVOID param)
	{
		SmartSockServer *server = (SmartSockServer*)param;
		if(server)
			server->Run();
		return 0;
	}

if(INVALID_SOCKET == (new_sockinfo.socket = accept(sockinfo.socket, (SOCKADDR*)&new_sockinfo.addr, &new_sockinfo.addr_size)))
            {
                return WSAGetLastError();
            }

mach doch mal testhalber 2 zeilen draus

new_sockinfo.socket = accept(sockinfo.socket, (SOCKADDR*)&new_sockinfo.addr, &new_sockinfo.addr_size);
            if(INVALID_SOCKET == new_sockinfo.socket)
            {
                return WSAGetLastError();
            }

denn WENN rein zufällig

INVALID_SOCKET == true auch true ergibt könnte eventuell dein programm bei

if((res = socket_accept(server, client)))
            return res;
        else

aussteigen weil WSAGetLastError() 0 zurückgibt wenn es keinen fehler gibt.

kleinen moment ....

(res = socket_accept(server, client)) ist immer true !!!
dein programm dürfte theoretisch gar nciht über das return res in deiner run() hinwegkommen

oder hab ich grad alles verpeilt ??
... oder wars INVALID_SOCKET ?? mist leider kann man beim AW schreiben die seiten nicht blättern um code zu kopieren

int SmartSockServer::Accept()
	{
		if(!socket_isvalid(server))
		{
			while(!socket_isvalid(server))
			{
				if(!socket_select(server))
				{
					EnterCriticalSection(&crit);
					CreateThread(NULL, NULL, ClientProc, this, NULL, NULL);
					MessageBox(NULL, "New thread", "Debug", NULL);
				}
			}
		}
		MessageBox(NULL, "OK, we lost the server", "Debug", NULL);
		return 1;
	}

if((res = socket_accept(server, client))) 
            return res; 
        else 
        { 
            LeaveCriticalSection(&crit); 

            socket_recv(client, buffer, 256, bytes); 
            buffer[bytes] = '\0'; 
            sscanf_s(buffer, "%i %s", &number, text); //hier oder
            if(number == 100) //hier hauts den karren an die wand, da steigt er aus wie es oben auf dem bild zu sehen ist

// CreateThread(NULL, NULL, ClientProc, this, NULL, NULL);
 ClientProc (this);

if(!socket_select(server))
				{
					//EnterCriticalSection(&crit);
					//CreateThread(NULL, NULL, ClientProc, this, NULL, NULL);
					ClientProc(this);
					MessageBox(NULL, "New thread", "Debug", NULL);
				}

in dem Bereich. Er kann auserdem nichts mehr an Sourcecode aufrufen, nur Disassembly

DWORD WINAPI ClientProc(LPVOID param)
{

 return 0;
}

Falls nicht, dann das gleiche nochmal mit :

DWORD WINAPI ClientProc(LPVOID param)
{
 SmartSockServer *server = new (SmartSockServer);

 delete (server); 

 return 0;
}

...

if(!socket_select(server))
{
 EnterCriticalSection(&crit);
 CreateThread(NULL, NULL, ClientProc, this, NULL, NULL);
// ClientProc(this);
 MessageBox(NULL, "New thread", "Debug", NULL);
} 

...

DWORD WINAPI ClientProc(LPVOID param)
{
 // erstmal "leer"
 return 0;
}

Wenns läuft, dann so :

...

DWORD WINAPI ClientProc(LPVOID param)
{
 SmartSockServer *server = (SmartSockServer*)param;
 if(server)
    server -> Run (); 
 return 0;
} 

...

int SmartSockServer::Run()
{
// erstmal "leer"

 MessageBox(NULL, "We leave te thread", "Debug", NULL);
 return 0;
}

int SmartSockServer::Run()
{
 SOCKINFO client;
 char buffer[256];
 char text[256];
 int res;
 int bytes;
 int number;

 if((res = socket_accept(server, client)))
  return res;
 else
 {
// hier "leer"
 }

 MessageBox(NULL, "We leave te thread", "Debug", NULL);
 return 0;
}

Wenns läuft dann :

...
 else
 {
  LeaveCriticalSection(&crit);

 }
...

Hab langsam das gefühl das eine string funktion nicht threadsafe ist, oder vlt doch was anderes?

Bin jetzt selbst soweit gekommen:

int SmartSockServer::Run()
	{
		SOCKINFO client;
		char buffer[256];
		char text[256];
		int res;
		int bytes;
		int number;

		if((res = socket_accept(server, client)))
			return res;
		else
		{
			LeaveCriticalSection(&crit);

			socket_recv(client, buffer, 256, bytes);
			buffer[bytes] = '\0';
			//sscanf_s(buffer, "%i %s", &number, text); //<--- ab der schmiert er ab im Debugbuild!!!

char buffer[256];
char text[256];
sscanf_s(buffer, "%i %s", &number, text, sizeof(text)); // <-- sizeof fehlte

und jetzt stürzt er vorerst nicht mehr ab ^^. Das sizeof() hab ich aber bisher noch nie wo gesehen... O_o

Verhindern lässt sich das durch sizeof () als letzten Parameter ?

Aha.

Prüfe unbedingt mal ob sprintf_s () auch solche Überraschungen bereithält.