Set exception whitelist check as earlier check instead of last check.

src/processor/exploitability_linux.cc

 // Copyright (c) 2013 Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 64 matching lines @@
         return EXPLOITABILITY_HIGH;
       }
 
       if (crashing_thread_frames[i]->function_name ==
           kBoundsCheckFailureFunction) {
         return EXPLOITABILITY_HIGH;
       }
     }
   }
 
-  // Check if the instruction pointer is in a valid instruction region
-  // by finding if it maps to an executable part of memory.
-  uint64_t instruction_ptr = 0;
-
   // Getting exception data. (It should exist for all minidumps.)
   MinidumpException *exception = dump_->GetException();
   if (exception == NULL) {
     BPLOG(INFO) << "No exception record.";
     return EXPLOITABILITY_ERR_PROCESSING;
   }
+  const MDRawExceptionStream *raw_exception_stream = exception->exception();
+  if (raw_exception_stream == NULL) {
+    BPLOG(INFO) << "No raw exception stream.";
+    return EXPLOITABILITY_ERR_PROCESSING;
+  }
+
+  // Checking for benign exceptions that caused the crash.
+  if (this->BenignCrashTrigger(raw_exception_stream)) {
+    return EXPLOITABILITY_NONE;
+  }
+
+  // Check if the instruction pointer is in a valid instruction region
+  // by finding if it maps to an executable part of memory.
+  uint64_t instruction_ptr = 0;
+
   const MinidumpContext *context = exception->GetContext();
   if (context == NULL) {
     BPLOG(INFO) << "No exception context.";
     return EXPLOITABILITY_ERR_PROCESSING;
   }
 
   // Getting the instruction pointer.
   if (!context->GetInstructionPointer(&instruction_ptr)) {
     return EXPLOITABILITY_ERR_PROCESSING;
   }
 
   // Checking for the instruction pointer in a valid instruction region.
   if (!this->InstructionPointerInCode(instruction_ptr)) {
     return EXPLOITABILITY_HIGH;
   }
 
-  const MDRawExceptionStream *raw_exception_stream = exception->exception();
-  if (raw_exception_stream == NULL) {
-    BPLOG(INFO) << "No raw exception stream.";
-    return EXPLOITABILITY_ERR_PROCESSING;
-  }
-
-  // Checking for benign exceptions that caused the crash.
-  if (this->BenignCrashTrigger(raw_exception_stream)) {
-    return EXPLOITABILITY_NONE;
-  }
-
   return EXPLOITABILITY_INTERESTING;
 }
 
 bool ExploitabilityLinux::InstructionPointerInCode(uint64_t instruction_ptr) {
   // Here we get memory mapping. Most minidumps will not contain a memory
   // mapping, so we will commonly resort to checking modules.
   MinidumpMemoryInfoList *mem_info_list = dump_->GetMemoryInfoList();
   const MinidumpMemoryInfo *mem_info =
       mem_info_list ?
       mem_info_list->GetMemoryInfoForAddress(instruction_ptr) : NULL;
@@ skipping 49 matching lines @@
     case MD_EXCEPTION_CODE_LIN_DUMP_REQUESTED:
       return true;
       break;
     default:
       return false;
       break;
   }
 }
 
 }  // namespace google_breakpad