Fix buffer overrun crash running MSVC2010 win32 Qt on win64
authorShane Kearns <ext-shane.2.kearns@nokia.com>
Thu, 23 Feb 2012 15:57:15 +0000 (15:57 +0000)
committerQt by Nokia <qt-info@nokia.com>
Thu, 1 Mar 2012 12:55:49 +0000 (13:55 +0100)
commit762a721f7f544f2428111f70f4d388f2d4134a55
tree6497d2aa17f6cdd9bb9c565eda8f876233851ea6
parent4908cf2a556948599c40681208438e814ba7b83c
Fix buffer overrun crash running MSVC2010 win32 Qt on win64

When loading dnsapi.dll as a plugin, we experience crashes because
the calling convention is not specified.
The default is _cdecl, but __stdcall (via the WINAPI macro) should
be used for windows APIs.
Mismatched calling convention results in corruption of local variables,
probably because the stack pointer is incorrect and SP offsets are
used in optimised builds rather than frame pointer offsets.

Since the library has been available since Windows 2000, I don't
think that we need to load it dynamically.
(Unlike the unix version where it isn't part of the LSB)

Also checked that the current release of mingw works.

Task-number: QTBUG-24227
Change-Id: I37c0a6aa0c133799c2a6dd9391ca1435ba2539ea
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
src/network/kernel/kernel.pri
src/network/kernel/qdnslookup_win.cpp