views:

152

answers:

3

Our unit tests fire off child processes, and sometimes these child processes crash. When this happens, a Windows Error Reporting dialog pops up, and the process stays alive until this is manually dismissed. This of course prevents the unit tests from ever terminating.

How can this be avoided?


Here's an example dialog in Win7 with the usual settings:

alt text

If I disable the AeDebug registry key, the JIT debugging option goes away:

alt text

If I disable checking for solutions (the only thing I seem to have control over via the control panel), it looks like this, but still appears and still stops the program from dying until the user presses something. WerAddExcludedApplication is documented to also have this effect.

alt text

+2  A: 

The only solution is to catch all exceptions at a very high level (for each thread) and terminate the application properly (or perform another action).

This is the only way to prevent the exception from escaping your app and activating WER.

Addition:

If the exception is something you do not except to happen you can use an AssertNoThrow(NUnit) or alike in another Unit Test framework to enclose the code firing the child processes. This way you would also get it into your Unit test report. This is in my opinion the cleanest possible solution I can think of.

Addition2: As the comments below show, I was mistaken: you cannot always catch the asynchronous exceptions, it depends on what the environment allows. In .NET some exceptions are prevented from being caught, what makes my idea worthless in this case...

For .NET: There are complicated workarounds involving the use of AppDomains, leading to an unload of an AppDomain instead of a crash of the whole application. Too bad...

http://www.bluebytesoftware.com/blog/PermaLink,guid,223970c3-e1cc-4b09-9d61-99e8c5fae470.aspx

http://www.develop.com/media/pdfs/developments_archive/AppDomains.pdf


EDIT:

I finally got it. With .NET 4.0 You can add the HandleProcessCorruptedStateExceptions attribute from System.Runtime.ExceptionServices to the method containing the try/catch block. This really worked! Maybe not recommended but works.

using System;
using System.Reflection;
using System.Runtime.InteropServices;
using System.Runtime.ExceptionServices;

namespace ExceptionCatching
{
    public class Test
    {
        public void StackOverflow()
        {
            StackOverflow();
        }

        public void CustomException()
        {
            throw new Exception();
        }

        public unsafe void AccessViolation()
        {
            byte b = *(byte*)(8762765876);
        }
    }

    class Program
    {
        [HandleProcessCorruptedStateExceptions]
        static void Main(string[] args)
        {
            Test test = new Test();
            try {
                //test.StackOverflow();
                test.AccessViolation();
                //test.CustomException();
            }
            catch
            {
                Console.WriteLine("Caught.");
            }

            Console.WriteLine("End of program");

        }

    }      
}
jdehaan
I suppose that would work in most cases, but what about StackOverflowException and AccessViolationException, which are uncatchable?
romkyns
Even these exceptions should be catchable. In C++ this might imply the use of asynchronous exceptions (`/EHa` flag for the MSVC++ compiler for example, see http://msdn.microsoft.com/en-us/library/1deeycx5.aspx). Becoming so technical, the question is not language agnostic anymore. In .NET you can catch these exceptions without tweaking around.
jdehaan
@jdehaan [I'm pretty sure](http://pastebin.com/paPChDsx) you [can't](http://msdn.microsoft.com/en-us/library/system.stackoverflowexception.aspx) in .NET. Not sure about C++. I'm trying to stay agnostic, but it seems your recommendation is dependent on language specifics...
romkyns
@romkyns, Ouch... In the very old past it worked in .NET, now not anymore. I understand the "technical reasons" behind, but sometimes you just have recursions and cannot control how deep they might become. I begin to dislike .NET regarding this... Thanks for the info, I really didn't know it until now!
jdehaan
Thanks for the snippets I experimented around... There seems to be a way so save the app from crashing with appdomains but that isn't funny. At least I learned a lot with your question. Unfortunately I cannot give more than a +1 to it, it is really really good.
jdehaan
@jdehaan Glad you enjoyed this question! Thanks for the link, that was a good read.
romkyns
I found another very interesting document located here: http://www.develop.com/media/pdfs/developments_archive/AppDomains.pdf. Funny that I didn't realize the behaviour was completely changed since .NET 2.0...
jdehaan
It worked with StackOverflow and Accessviolation now but not with the code from http://stackoverflow.com/questions/2950130/how-to-simulate-a-corrupt-state-exception-in-net-4. At least a bit of progress.
jdehaan
Great find! Didn't know of that one.
romkyns
Did you have success with the AppDomain workaround? for me this doesn't work for these exceptions, not only the AppDomain is unloaded but the whole Application was torn down! Is the AppDomain separation a myth? :-) Unfortunately I will have less time to dig into that next weeks. I hope I could help a bit. I also strongly want to understand this to 100%.
jdehaan
@jdehaan Indeed, the AppDomain isolation doesn't appear to be quite as good as the process isolation. By the way, +1 on your answer and I've posted a community wiki answer of my own summarizing all the answers I've found.
romkyns
+2  A: 

Try setting

HKCU\Software\Microsoft\Windows\Windows Error Reporting\DontShowUI

to 1. (You can also set the same key in HKLM, but you need admin privs to do that.)

This should prevent WER from showing any UI.

Eric Brown
+1  A: 

A summary from the answers by jdehaan and Eric Brown, as well as this question:

Option 1:

Works globally on the entire user account or machine, which can be both a benefit and a drawback.

Set [HKLM|HKCU]\Software\Microsoft\Windows\Windows Error Reporting\DontShowUI to 1. More info: WER settings.

Option 2:

Requires modification to the crashing program.

Call SetErrorMode: SetErrorMode(SetErrorMode(0) | SEM_NOGPFAULTERRORBOX);. More info: Disabling the program crash dialog (explains the odd arrangement of calls).

Option 3:

Requires modification to the crashing program.

Use SetUnhandledExceptionFilter to set your own structured exception handler that simply exits.

Option 4:

Requires modification to the crashing program. For .NET applications only.

Wrap all code into a global try/catch block. Specify the HandleProcessCorruptedStateExceptionsAttribute and possibly also the SecurityCriticalAttribute on the method catching the exceptions. More info: Handling corrupted state exceptions

Option 5:

Works globally on the entire user account, but only for a controlled duration.

Kill the Windows Error Reporting process whenever it shows up:

var werKiller = new Thread(() =>
{
    while (true)
    {
        foreach (var proc in Process.GetProcessesByName("WerFault"))
            proc.Kill();
        Thread.Sleep(3000);
    }
});
werKiller.IsBackground = true;
werKiller.Start();

This is not completely bullet-proof though, because a console application may crash via a different error message, apparently displayed by an internal function called NtRaiseHardError:

alt text

romkyns