Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(579)

Side by Side Diff: chrome_frame/exception_barrier.cc

Issue 1733021: Add an ExceptionBarrier around outbound calls to patched methods in IE. In so... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
Property Changes:
Added: svn:eol-style
+ LF
OLDNEW
(Empty)
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4 //
5 // A class to make it easy to tag exception propagation boundaries and
6 // get crash reports of exceptions that pass over same.
7 #include "chrome_frame/exception_barrier.h"
8
9 enum {
10 // Flag set by exception handling machinery when unwinding
11 EH_UNWINDING = 0x00000002
12 };
13
14 ExceptionBarrier::ExceptionHandler ExceptionBarrier::s_handler_ = NULL;
15
16 // This function must be extern "C" to match up with the SAFESEH
17 // declaration in our corresponding ASM file
18 extern "C" EXCEPTION_DISPOSITION __cdecl
19 ExceptionBarrierHandler(struct _EXCEPTION_RECORD *exception_record,
20 void * establisher_frame,
21 struct _CONTEXT *context,
22 void * reserved) {
23 establisher_frame; // unreferenced formal parameter
24 reserved;
25 if (!(exception_record->ExceptionFlags & EH_UNWINDING)) {
26 // When the exception is really propagating through us, we'd like to be
27 // called before the state of the program has been modified by the stack
28 // unwinding. In the absence of an exception handler, the unhandled
29 // exception filter gets called between the first chance and the second
30 // chance exceptions, so Windows pops either the JIT debugger or WER UI.
31 // This is not desirable in most of the cases.
32 ExceptionBarrier::ExceptionHandler handler = ExceptionBarrier::handler();
33 if (handler) {
34 EXCEPTION_POINTERS ptrs = { exception_record, context };
35
36 handler(&ptrs);
37 }
38 }
39
40 return ExceptionContinueSearch;
41 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698