This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] On RTEMS, avoid recursion when reporting Mmap failure
ClosedPublic

Authored by waltl on May 4 2018, 1:49 PM.

Reviewers

Commits

Repository

rL LLVM

Build Status

Buildable 17722
Build 17722: arc lint + arc unit

LGTM

This revision is now accepted and ready to land.May 4 2018, 3:01 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

compiler-rt/

lib/

sanitizer_common/

7 lines

	Show All 32 Lines
	// Cached pid of parent process - if the parent process dies, we want to keep			// Cached pid of parent process - if the parent process dies, we want to keep
	// writing to the same log file.			// writing to the same log file.
	uptr stoptheworld_tracer_ppid = 0;			uptr stoptheworld_tracer_ppid = 0;

	void NORETURN ReportMmapFailureAndDie(uptr size, const char *mem_type,			void NORETURN ReportMmapFailureAndDie(uptr size, const char *mem_type,
	const char *mmap_type, error_t err,			const char *mmap_type, error_t err,
	bool raw_report) {			bool raw_report) {
	static int recursion_count;			static int recursion_count;
	if (raw_report \|\| recursion_count) {			if (SANITIZER_RTEMS \|\| raw_report \|\| recursion_count) {
	// If raw report is requested or we went into recursion, just die.			// If we are on RTEMS or raw report is requested or we went into recursion,
	// The Report() and CHECK calls below may call mmap recursively and fail.			// just die. The Report() and CHECK calls below may call mmap recursively
				// and fail.
	RawWrite("ERROR: Failed to mmap\n");			RawWrite("ERROR: Failed to mmap\n");
	Die();			Die();
	}			}
	recursion_count++;			recursion_count++;
	Report("ERROR: %s failed to "			Report("ERROR: %s failed to "
	"%s 0x%zx (%zd) bytes of %s (error code: %d)\n",			"%s 0x%zx (%zd) bytes of %s (error code: %d)\n",
	SanitizerToolName, mmap_type, size, size, mem_type, err);			SanitizerToolName, mmap_type, size, size, mem_type, err);
	#if !SANITIZER_GO			#if !SANITIZER_GO
	▲ Show 20 Lines • Show All 313 Lines • Show Last 20 Lines