C / C++ / MFC

Re: UDP Application on Vista: Fail on sendto function

17-Sep-07 10:20

Hello,

I have UDP app that works fine, except on Windows Vista.
The error is WSAEINVAL (10022) on sendto function, and the package did not send Frown | :-(

Anyone know this problem?

Thanks,
Cris.

Mark Salsbery17-Sep-07 11:32

Re: UDP Application on Vista: Fail on sendto function

17-Sep-07 11:32

Tough to guess without knowing the code Smile | :)

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

David Crow17-Sep-07 11:56

David Crow

17-Sep-07 11:56

Cris wrote:
The error is WSAEINVAL (10022) on sendto function...

WSAEINVAL = "An unknown flag was specified, or MSG_OOB was specified for a socket with SO_OOBINLINE enabled."

"A good athlete is the result of a good and worthy opponent." - David Crow

"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne

Re: UDP Application on Vista: Fail on sendto function

Cris18-Sep-07 3:53

Re: UDP Application on Vista: Fail on sendto function

18-Sep-07 3:53

This is my code:

<br />
SOCKET m_maSocketOut;<br />
struct sockaddr_in m_SockAddrOut;<br />
unsigned long m_lInet;<br />
.<br />
.<br />
<br />
void init(CString strServer)<br />
{<br />
unsigned long lInet=0;<br />
<br />
memset(&m_SockAddrOut, 0, sizeof(m_SockAddrOut));<br />
<br />
m_maSocketOut = socket(PF_INET, SOCK_DGRAM, IPPROTO_UDP); <br />
m_SockAddrOut.sin_family = AF_INET;<br />
m_SockAddrOut.sin_port = htons(port);<br />
<br />
struct hostent FAR* hp = gethostbyname((char FAR *) (const char *) strServer);<br />
<br />
if (hp == NULL)<br />
{<br />
   lInet = inet_addr((const char * ) strServer); <br />
   if (lInet != INADDR_NONE)	<br />
   {<br />
      validIP = TRUE;		<br />
      byAddress = TRUE;		<br />
   }<br />
}<br />
else<br />
   validIP = TRUE;<br />
	<br />
if (validIP)<br />
{<br />
   if (byAddress)<br />
   {<br />
      m_SockAddrOut.sin_addr.s_addr = lInet;<br />
      m_lInet = lInet;<br />
   }<br />
   else<br />
   {<br />
      m_SockAddrOut.sin_addr.s_addr = *((unsigned long far *) hp->h_addr);<br />
      m_lInet = *((unsigned long far *) hp->h_addr);<br />
   }<br />
<br />
   if (m_maSocketOut != NULL)<br />
   {<br />
      bind(m_maSocketOut, (SOCKADDR*)& m_SockAddrOut, sizeof(m_SockAddrOut));<br />
      WSAAsyncSelect(m_maSocketOut, hWnd, WM_EXTERNA, FD_READ | FD_CLOSE);<br />
   }<br />
}<br />
}<br />
<br />
void sendData(char szData[], int len)<br />
{<br />
   int result = sendto(m_maSocketOut, (char FAR *) &szData, len, <br />
		    NULL, (PSOCKADDR) &m_SockAddrOut, sizeof(m_SockAddrOut));<br />
   m_SockAddrOut.sin_addr.s_addr = INADDR_ANY;<br />
<br />
   if (result < 0)<br />
   {<br />
      int erroCode = WSAGetLastError();<br />
      //erroCode = 10022 in Windows Vista!!<br />
   }<br />
}<br />

Important: This error code is only on Windows Vista, but not all instalations . I don't know which the exat situation, but I observed in Business and Ultimate version.

Any idea?

David Crow18-Sep-07 3:58

David Crow

18-Sep-07 3:58

Could it be a UAC thing?

"A good athlete is the result of a good and worthy opponent." - David Crow

"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne

Re: UDP Application on Vista: Fail on sendto function

Cris18-Sep-07 4:44

Re: UDP Application on Vista: Fail on sendto function

18-Sep-07 4:44

No, the UAC configuration is not enabled.

Cris20-Sep-07 3:28

20-Sep-07 3:28

I have solved the problem... removing the bind() function execution.

Acording with the MSDN documentation:

"The bind function is used on an unconnected socket before subsequent calls to the connect or listen functions. It is used to bind to either connection-oriented (stream) or connectionless (datagram) sockets. When a socket is created with a call to the socket function, it exists in a namespace (address family), but it has no name assigned to it. Use the bind function to establish the local association of the socket by assigning a local name to an unnamed socket."

I understood that the bind function is not necessary, because I only want to send datagrams with a SOCKET to server and receive the responses by the same SOCKET, and the WSAAsyncSelect function grants the data response to the app.

This is correct?

Thanks,
Cris.

HTML help integration

Nathan Holt at EMOM17-Sep-07 9:41

17-Sep-07 9:41

I've been looking through the MSDN documentation, but I haven't found how to add topics to html help in a VS2003 project. In particular, some how dialog controls must be maped to IDs in the HTML help project file. (I found how to get from there to the html files.) Could anyone explain how this happens?

Nathan

Failed new operator

Joe Smith IX17-Sep-07 9:03

17-Sep-07 9:03

Hi all,

I am trying to use this class http://www.codeproject.com/cs/files/CABCompressExtract.asp?df=100&forumid=336576&fr=26[^] in my dialog class. But as soon as I declare the object, it failed to compiled with the following error:

c:\testcompress\compresst.hpp(466) : error C2661: 'new' : no overloaded function takes 2 parameters
        c:\testcompress\compresst.hpp(465) : while compiling class-template member function 'void *__cdecl Cabinet::CCompressT<class Cabinet::CCompress>::Alloc(unsigned int)'
Error executing cl.exe.

Here is my function call:

void CTestCompressDlg::OnButton1() 
{
  // I copied these lines from the demo source code
  UINT u32_CabID = 12345;
  UINT u32_SplitSize = 0x7FFFFFFF;
 
  char s8_CompressFile[MAX_PATH];
 
  Cabinet::CCompress        i_Compress;
  i_Compress.CreateFCIContext(s8_CompressFile, u32_SplitSize, u32_CabID);
 
}

And here is the snippet on the failing line:

namespace Cabinet
{

template <class T> class CCompressT
{
public:
...

protected:
  static void* Alloc(UINT size)
  { 
    return operator new(size); // <--- failed here
  }

...
}
}

Anyone has any idea why, please? Thanks a lot in advance.

Mark Salsbery17-Sep-07 9:21

17-Sep-07 9:21

Maybe try

return new BYTE(size);

or

return new unsigned char(size);

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Joe Smith IX17-Sep-07 9:25

17-Sep-07 9:25

They didn't work either.

Btw, the demo compiles fine. How could it be?

Mark Salsbery17-Sep-07 9:32

17-Sep-07 9:32

Same compiler error? It works for me on VS 2005.

Do you need to include new.h?

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Joe Smith IX17-Sep-07 9:42

17-Sep-07 9:42

Hi Mark,

Just to be clear, this is what I did:
1. Create a dialog based app
2. #include "ExtractResourceT.hpp"
#include "CompressT.hpp"
3. Create a button with OnButton1() function as above
4. Compile

Are you telling me that those steps works in VS2005?
(Mine is VC++ 6).

PS. "include new.h" generated even more errors relating to new.h

Nathan Holt at EMOM17-Sep-07 9:35

17-Sep-07 9:35

I recall that Visual C++ generates code with a debug feature in which new is #Defined as something else in debug builds. Does it compile in release mode?

Nathan

Joe Smith IX17-Sep-07 9:47

17-Sep-07 9:47

You are right. It compiles fine in release mode. This is new and weird to me...

How am I supposed to debug my app then?

Nathan Holt at EMOM17-Sep-07 10:24

17-Sep-07 10:24

Joe Smith IX wrote:
You are right. It compiles fine in release mode. This is new and weird to me...

How am I supposed to debug my app then?

In the implementation file you are using, you could look for the #define new statement and remove it. If you want to keep the ability to detect memory leaks from that code, you can change it to #define TNEW, add a #else clause with #define TNEW new, and then replace all standard calls to new with TNEW.

Nathan

Joe Smith IX17-Sep-07 16:34

17-Sep-07 16:34

But when I search for it, the only two instances of 'define new' are the standard one below in my dialog and app .cpp files. Could it be defined some other way? Or any other workaround to make it compilable in debug mode?
Thanks.

#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif

Mark Salsbery17-Sep-07 18:28

17-Sep-07 18:28

Do you really need to call the operator directly?

Can you just use the vector form of new and allocate an array of bytes?

return new BYTE[size];

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Joe Smith IX17-Sep-07 21:21

17-Sep-07 21:21

Well, the thing is that the call is made from the class I downloaded from CP (see my first post above). If I change the object, then I have to modify the class as well, right?

What makes me wonder is that how come I can run the demo code in debug mode and step thru line-by-line, while in my test project I can't even compile the debug mode? They both using the same classes!

Any ideas? thanks for any suggestion.

Mark Salsbery18-Sep-07 5:16

18-Sep-07 5:16

If the problem is with using the DEBUG_NEW operator in debug builds, then you
can wrap your calls to operator new with something like

#pragma push_macro("new")
#undef new
...
(use the new operator here)
...
#pragma pop_macro("new")

The problem is, where do you put this in a template class? Do you put it in the class
declaration or do you have to remember to do it every place you instantiate an object
of the class?

I haven't seen the rest of the class in question, but I assume if it calls operator new, then
it must call operator delete somewhere. Those should be the only calls you'd need to change.

I have no idea why the author did that - is the operator new overloaded in the class?

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Nathan Holt at EMOM18-Sep-07 7:29

18-Sep-07 7:29

Mark Salsbery wrote:
#pragma push_macro("new")
#undef new
...
(use the new operator here)
...
#pragma pop_macro("new")

That's cool. I didn't know about the #pragma push_macro. Thanks for mentioning it.

Nathan

Joe Smith IX18-Sep-07 7:47