[UR][ASAN] Possible `nullptr` dereference

[AlexeySachkov]

llvm/unified-runtime/source/loader/layers/sanitizer/asan/asan_interceptor.cpp

Lines 667 to 668 in 8c62c24

	auto PI = getProgramInfo(Program);
	bool IsInstrumented = PI->isKernelInstrumented(Kernel);

In the snippet above PI is a shared_ptr which could be nullptr:

llvm/unified-runtime/source/loader/layers/sanitizer/asan/asan_interceptor.hpp

Lines 334 to 340 in 8c62c24

	std::shared_ptr<ProgramInfo> getProgramInfo(ur_program_handle_t Program) {
	std::shared_lock<ur_shared_mutex> Guard(m_ProgramMapMutex);
	if (m_ProgramMap.find(Program) != m_ProgramMap.end()) {
	return m_ProgramMap[Program];
	}
	return nullptr;
	}

Perhaps we don't ever expect it to be nullptr and there is no issue, but in that case we should add an assert statement to asan_interceptor.cpp to indicate that.

[kbenzie]

@cdai2 could you triage this please?

[kbenzie]

Tracked in URT-918

[yingcong-wu]

Yes, that issue has been brought to our attention, and we will work on this issue later. Thanks for reporting.