Fixes to possible GetLastError bugs

base/debug/stack_trace_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/debug/stack_trace.h"
 
 #include <windows.h>
 #include <dbghelp.h>
 
 #include <iostream>
@@ skipping 59 matching lines @@
   // add the directory of the executable to symbol search path.
   // All following errors are non-fatal.
   const size_t kSymbolsArraySize = 1024;
   scoped_ptr<wchar_t[]> symbols_path(new wchar_t[kSymbolsArraySize]);
 
   // Note: The below function takes buffer size as number of characters,
   // not number of bytes!
   if (!SymGetSearchPathW(GetCurrentProcess(),
                          symbols_path.get(),
                          kSymbolsArraySize)) {
+    g_init_error = GetLastError();
     DLOG(WARNING) << "SymGetSearchPath failed: " << g_init_error;
-    g_init_error = GetLastError();
     return false;
   }
 
   std::wstring new_path(std::wstring(symbols_path.get()) +
                         L";" + GetExePath().DirName().value());
   if (!SymSetSearchPathW(GetCurrentProcess(), new_path.c_str())) {
     g_init_error = GetLastError();
     DLOG(WARNING) << "SymSetSearchPath failed." << g_init_error;
     return false;
   }
@@ skipping 184 matching lines @@
       (*os) << "\t" << trace_[i] << "\n";
     }
   } else {
     (*os) << "Backtrace:\n";
     context->OutputTraceToStream(trace_, count_, os);
   }
 }
 
 }  // namespace debug
 }  // namespace base