C(++) Sockets och Threads

Håller på att göra en multitrådad webserver, för varje socket accept jag gör ska jag skapa en tråd som jag skickar in i en tråd, funkar skit fint utan trådar men när jag väl skickar med socketen till en ny tråd verkar det som att den förlorar alla sin värden eller något sådant, får lite skumma errors. Jag gör förmodligen något fel, postar relevanta delar av koden.

Hur det ser ut i main:

struct sockaddr clientAddr;
		int clientAddrLen = sizeof(clientAddr);
		SOCKET s1 = accept(s,&clientAddr,&clientAddrLen);

		if(s1 != INVALID_SOCKET) {

			_beginthread(Server,0,(void*)&s1);
		}

och sedan hur det ser ut på "Server" trådens början då jag typar om den och tar emot meddelandet, som bara blir en tom sträng...

void Server( void *s) {

	SOCKET s1 = *(SOCKET*)s;
	char hostName[100];
	char portName[100];
	time_t lt;
	lt = time(NULL);

	// Vänta på inkommande anrop
	struct sockaddr clientAddr;
	int clientAddrLen = sizeof(clientAddr);
	if(s1 != INVALID_SOCKET) {

		int ok = getnameinfo(&clientAddr,clientAddrLen,hostName,100,portName,100,NI_NUMERICSERV );

		if(ok == 0) {
			printf("Accept incoming from: %s at port %s\n",hostName,portName);
		}
	}

	// Skriv ut meddelandet från klienten, remade
	int iResult;
	char inputHTTP[512] = {0};
	iResult = recv(s1, inputHTTP, 512, 0);

Har sökt runt på detta ganska länge nu och alla exempel ser exakt likadant ut för mig men... blir bara kalle kalas av det hela :s även variablerna "hostName" och "portName" blir bara skräpvärden...

har också prövat med CreateThread() men samma sak där, all hjälp uppskattas tackar!

Prövade nu
_beginthread(Server,0,(void*)s1);

och

SOCKET s1 = (SOCKET)s;

Castar om fint osv, men socketen som kommer fram har samma värde men när jag kör recv för att få meddelandet och så vidare.
Kan gärna skicka allt, det är för en skolgrej suttit med detta i tre dagar snart, så har verkligen givit det mitt allt.

dannne1@hotmail.com @msn

getnameinfo(&clientAddr,clientAddrLen,hostName,100,portName,100,NI_NUMERICSERV );

får jag bara massa skräp

Tack förresten för tipsen, försöker vidare

Säg till här om ni har fler tips eller ideer

Inte så mycket info där, de länkarna som fanns i andra tråden har jag redan kollat på och sedan ska detta arbetet vara ganska enkelt så ska inte behöva skapa egna klasser för det. Sedan ska det vara gjort i C så mycket som möjligt, inte jobbat i C på länge men inte objektorienterat?

Tack, följer dock hans tråd och ser ifall det finns något jag kan snappa upp!

Alltså projektet skall inte vara så stort, har inte en aning om hur jag gör en struct, meningen är att man ska vara 5 personer på detta och detta är egentligen en liten del av laborationen, bara det att alla mina gruppmedlemmar har hoppat av så jag är ensam kvar :/ mindre roligt.

Men nu förstår jag vad som är felet, jag måste på något sätt som du säger skicka med värdena som skapas i accept.

Märkte nu att Recv har fixats efter att jag följde tidigare råd! en liten bit på vägen men nu kan jag sova lugnare!

Tackar på förhand, detta hjälpte mig verkligen, men nu stupar jag snart, suttit med detta sedan 4 i eftermiddags. Natti natt!

Har du något bra exempel på detta? Vore snällt med relativt snabbt svar, uppgiften skall vara klar till ikväll och tänkte åtminstone ha trådarna klara tills dess, sedan får det bli komplettering på resten så länge

getpeerinfo borde räcka vad jag kan se, ger mig hostname och port som jag är ute efter, får den dock inte att fungera...

Tack igen!

EDIT: Prövade nu att skapa en struct efter att läst lite guider, får nu error på raderna:

_beginthread(Server,0,(void*)conn);

och

Connections conn = (Connections)s;

Den första raden i min main då där jag skickar min struct "conn" och andra i tråden när jag typecastar om void* till min struct som heter Connections.

Felet jag får är "Cannot convert from connections to void* och vice versa för andra raden... :s

Prövade nu att skapa en struct efter att läst lite guider, får nu error på raderna:

_beginthread(Server,0,(void*)conn);

och

Connections conn = (Connections)s;

Den första raden i min main då där jag skickar min struct "conn" och andra i tråden när jag typecastar om void* till min struct som heter Connections.

Felet jag får är "Cannot convert from connections to void* och vice versa för andra raden... :s

Prövar nu getpeername som du föreslod

struct sockaddr_in clientAddr;
socklen_t clientAddrLen = sizeof(clientAddr);
if (getpeername(s1, &clientAddr, &clientAddrLen) == -1) {
    perror("getpeername");
    /* Perhaps return? */
}

if (getnameinfo(&clientAddr, clientAddrLen, other arguments ...) == -1) {
    perror("getnameinfo");
    /* Return? */
}

	struct sockaddr_in clientAddr;
	socklen_t clientAddrLen = sizeof(clientAddr);
	getpeername(s1, &clientAddr, &clientAddrLen);
	getnameinfo(&clientAddr, clientAddrLen,hostName,100,portName,100,NI_NUMERICSERV);

Kör jag nu då, men får typecast fel då...

'getpeername' : cannot convert parameter 2 from 'sockaddr_in *' to 'sockaddr *'

och

'getnameinfo' : cannot convert parameter 1 from 'sockaddr_in *' to 'const SOCKADDR *'

Så den gillar inte riktigt sockaddr_in som du använt där, jag vet heller inte riktigt vad det är.. inte hemma på detta direkt

_beginthread(Server,0,(void*)conn);

och

Connections conn = (Connections)s;

Löste jag halvt genom att skriva

_beginthread(Server,0,(void*)&conn);

fast

Connections conn = (Connections)s;

får jag fortf fel på, hur typecastar jag en struct ordentlige med pekare osv?

Denna lösningen verkar lättare tycker jag, om det är som jag tror

Får inte skapa klasser för denna labben enligt speccen ifrån läraren :/

okok, testar detta:

Så här börjar tråden:

SOCKET s1 = (SOCKET)s;
	char hostName[100];
	char portName[100];


	struct sockaddr_in clientAddr;
	socklen_t clientAddrLen = sizeof(clientAddr);
	int a = getpeername(s1, (struct sockaddr*)&clientAddr, &clientAddrLen);
	int k = getnameinfo((struct sockaddr*)&clientAddr, clientAddrLen,hostName,100,portName,100,NI_NUMERICSERV);

int iResult;
	char inputHTTP[512] = {0};
	iResult = recv(s1, inputHTTP, 512, 0);

Får fortfarande bara massa skräpvärden i mina portName och hostName variabler, märker idag också att recv inte fungerar, alltså socketen som jag skickar verkar vara helt värdelös, int a blir 0 och int k blir 10093

Så här ser det ut i main där jag startar tråden:

SOCKET s1 = accept(s,&clientAddr,&clientAddrLen);

		if(s1 != INVALID_SOCKET) {
			_beginthread(Server,0,(void*)s1);
		}

Förslag?

PS: Kör jag med felhanteringen du föreslod får jag bara outputen: "getpeername: no error" :s

GAAAAHHHH okey, tackar för hjälpen alla min trådning funkar nu och recv funkar också, fick dock massa fel, mest dock för att jag körde:

closesocket(s);
WSACleanup();

i main... dock hade var det dock inte det som förstörde mina
getpeername
och
getnameinfo

men nu funkar hela trådningen, skall fixa ett telnet admin gränssnitt också med vissa kommandon, fast det får bli komplettering på det.

Tackar igen för hjälpen! varit skit bra!