[Bug 24180] New: crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Summary: crash in dnsbench.exe
           Product: Wine
           Version: 1.3.1
          Platform: x86
               URL: http://www.grc.com/dev/r-and-d/1.1.3891.0.exe
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=30436)
 --> (http://bugs.winehq.org/attachment.cgi?id=30436)
terminal.txt

DNSBench.exe which benchmarks and checks DNS is currently under development.
The latest release crashes on exit. Might be related to these messages
occurring just before the crash:

dnsapi:dns_map_options option DNS_QUERY_BYPASS_CACHE not implemented
followed by:
err:ntdll:RtlpWaitForCriticalSection 0x2016a940 "query.c: resolver_cs" wait
timed out

wine-1.3.1-135-g2c127dd

How to repeat:
To see this error download the latest version in the URL, run it, wait until
the spinning G stops spinning as it checks internet connectivity and a few
things and then closes. Seems to happen regularly except if user does not wait
for the spinning 'G' to stop.

The developer commented:

The dump says that the crash is occurring inside the DnsQuery 
call of the DnsAPI. DNSB began using that DLL library for the 
DNS rebinding attack protection detection.

FWIW
DnsQuery Function:
http://msdn.microsoft.com/en-us/library/ms682016%28VS.85%29.aspx
DNS Query Options:
http://msdn.microsoft.com/en-us/library/cc982162%28v=VS.85%29.aspx

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

--- Comment #1 from [e-mail] 2010-08-28 02:06:16 CDT ---
whups: executable file download is 164Kb (written in assembly).

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from [e-mail] 2010-08-28 02:36:00 CDT ---
CONFIRMED: Renaming dnsapi.dll.so eliminates the related console spam.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

--- Comment #3 from [e-mail] 2010-08-28 02:39:15 CDT ---
Created an attachment (id=30437)
 --> (http://bugs.winehq.org/attachment.cgi?id=30437)
Console output.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

--- Comment #4 from Hans Leidekker  2010-08-28 05:07:23 CDT ---
Created an attachment (id=30440)
 --> (http://bugs.winehq.org/attachment.cgi?id=30440)
dnsapi: Remove resolver lock.

Please try this patch.

The app runs many queries in separate threads and it looks like one
of them gets killed before the query completes, while still holding
the resolver lock.

This could be part of normal operation, so our best option may be to
remove the critical section and rely on thread safety of res_query.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Hans Leidekker  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #5 from Hans Leidekker  2010-08-28 06:27:09 CDT ---
Looks like glibc's resolver supports threads since 2.3.2, released in 2003.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

--- Comment #6 from [e-mail] 2010-08-28 06:47:29 CDT ---
(In reply to comment #5)
> Looks like glibc's resolver supports threads since 2.3.2, released in 2003.

that seems to have done it - Cant make it crash now.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

--- Comment #7 from Hans Leidekker  2010-09-08 14:13:24 CDT ---
Fixed by 18cd143086ef040f9322b0b9b017cb47a0a6946f.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #8 from Austin English  2010-09-08 16:23:14 CDT ---
Fixed.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #9 from Alexandre Julliard  2010-09-18 13:05:09 CDT ---
Closing bugs fixed in 1.3.3.

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |1.2.x

[Bug 24180] crash in dnsbench.exe

http://bugs.winehq.org/show_bug.cgi?id=24180

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|1.2.x                       |---

--- Comment #10 from Alexandre Julliard  2010-10-08 10:40:23 CDT ---
Removing 1.2.x milestone from bugs fixed in 1.2.1.