mirror of
https://github.com/chromium/crashpad.git
synced 2024-12-28 15:50:26 +08:00
9c1719802f
Previous commits[1][2] modified tests to require looking at RawLog output to validate the success or failure of the iOS InHandlerProcess. Previously this would use freopen to direct the RawLog to a file. However, freopen introduces a race where the log file may not be associated with stderr, and instead may interfere with the InProcessHandler's cached writer fd. This caused flake with the intermediate dumps sometimes including stderr logging. Since the test fixtures only needs to know about the output of RawLog, instead add a crashpad::internal::SetFileHandleForTesting method to that swaps out STDERR_FILENO with the test fixture's fd. [1] https://crrev.com/c/3488826 [2] https://crrev.com/c/3401563 Change-Id: I87b1020db6b896a47bec5a7c916a572c192b884f Reviewed-on: https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3517773 Reviewed-by: Joshua Peraza <jperaza@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org>
45 lines
1.6 KiB
C++
45 lines
1.6 KiB
C++
// Copyright 2021 The Crashpad Authors. All rights reserved.
|
||
//
|
||
// Licensed under the Apache License, Version 2.0 (the "License");
|
||
// you may not use this file except in compliance with the License.
|
||
// You may obtain a copy of the License at
|
||
//
|
||
// http://www.apache.org/licenses/LICENSE-2.0
|
||
//
|
||
// Unless required by applicable law or agreed to in writing, software
|
||
// distributed under the License is distributed on an "AS IS" BASIS,
|
||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||
// See the License for the specific language governing permissions and
|
||
// limitations under the License.
|
||
|
||
#ifndef CRASHPAD_UTIL_IOS_EXCEPTION_LOGGING_H_
|
||
#define CRASHPAD_UTIL_IOS_EXCEPTION_LOGGING_H_
|
||
|
||
#include "util/file/file_io.h"
|
||
|
||
namespace crashpad {
|
||
namespace internal {
|
||
|
||
//! \brief Log \a message to stderr in a way that is safe to run during an
|
||
//! in-process crash. Also prints the given file, line number and an
|
||
//! optional error code.
|
||
//!
|
||
//! Note: RUNS-DURING-CRASH.
|
||
void RawLog(const char* file, int line, const char* message, int error);
|
||
|
||
//! \brief Direct RawLog to log to \a file_handle instead of stderr, so tests
|
||
//! can confirm certain error conditions during in-process crashes. Call
|
||
//! before before any Crashpad is run.
|
||
void SetFileHandleForTesting(FileHandle file_handle);
|
||
|
||
} // namespace internal
|
||
} // namespace crashpad
|
||
|
||
#define CRASHPAD_RAW_LOG(message) \
|
||
::crashpad::internal::RawLog(__FILE__, __LINE__, message, 0)
|
||
|
||
#define CRASHPAD_RAW_LOG_ERROR(error, message) \
|
||
::crashpad::internal::RawLog(__FILE__, __LINE__, message, error)
|
||
|
||
#endif // CRASHPAD_UTIL_IOS_EXCEPTION_LOGGING_H_
|