Thread: winsock and dialogboxes

Well given that WSAStartup only needs to be called once (you can call it many times, but you need to match it with the same number of WSACleanup calls), I'd say calling it every time in a dialog proc is bad news.

try:

Code:

....
#include<winsock.h>
....

BOOL CALLBACK ServerDlgProc(HWND hwnd, UINT Message, WPARAM wParam, LPARAM lParam)
{

WORD wVersionRequested = MAKEWORD(1,1);
    WSADATA wsaData;
    static int SVR = WSAStartup(wVersionRequested, &wsaData);

	switch(Message)
	{
		case WM_INITDIALOG:

		return TRUE;

		case WM_COMMAND:
			switch(LOWORD(wParam))
			{

                     case IDC_SRVR:
                             {

....

also it might not even be a socket related problem, make sure that there are appropriate breaks/returns/gotos in each case (unless you want an intentional fall through, that is handle the cases with the same code) so that execution is not falling through to whatever case closes the window.

Lemme guess, the 'static' hack mentioned by sl34k ?

How about connecting to winsock with the first message (which matters) when the dialog proc is invoked.

So you are saying put it in the winproc under wm_command ?

Code:

		case WM_COMMAND:
			switch(LOWORD(wParam))
			{

where the MAKEINTRESOURCE( is for the dialog box. there is good tutes on winsock and good tutes on winapi but not on combininding the two. this with the dialogboxs is a different way than i am use too. ususally it is just one winmain and no dbs.

will give that a try.

eidt going back and reading http://tangentsoft.net/wskfaq/

Well normally, since you only need one WSAStartup per process, putting it close to the start of winMain() seems to be normal practice, so that you can call WSACleanup just the once when you finally get around to ending the program.

Or are you trying to defer network activity until absolutely necessary.

Or are you trying to defer network activity until absolutely necessary.

exactly. i thought i could put all winsock activity in a dialogbox and that way only call it when needed. was trying to save ram and only when needed sock activity. ok will put in winmain(). was converting console sock prog to win.

also need to convert argv[] to lpcstr.

Code:

main(int argc, char **argv)
{
    WORD wVersionRequested = MAKEWORD(1,1);
    WSADATA wsaData;

to edit box     EDITTEXT        IDC_TEXTB,...., ES_AUTOVSCROLL | ES_MULTILINE | ES_WANTRETURN from .rc file. 

from dialog srvrproc....
 buff = (char*)GlobalAlloc( GPTR, textlen + 1);
                                                     GetDlgItemText(hwnd,IDC_TEXTB, buff , textlen + 1);

to sock
int Printsrvr(LPCSTR lpServerNameOrAddress)
{....

    iaHost.s_addr = inet_addr(lpServerNameOrAddress);

instead of getting **argv from console would be getting webpage from dialogbox idc_textb editbox to buff then convert to lpcstr lpservernameoraddr hence the prob of where to put sock. page not always needed. hope that made sense. again thank you.

edit convert globalalloc buff to lpcstr lpaddr....