million/suyu
1
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

nvhost_ctrl_gpu: Avoid sending null pointer to memcpy

Browse source 
Undefined Behaviour Sanitizer reports a null pointer is being sent to
memcpy, thought it's "guaranteed to never be null". Guard it with an if
statement, and log when the action has been averted.
This commit is contained in:
lat9nq 2021-04-04 15:39:38 -04:00
parent 69b2dbdffd
commit 638c892edf
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/core/hle/service/nvdrv/devices/nvhost_ctrl_gpu.cpp
View file

@ -248,7 +248,13 @@ NvResult nvhost_ctrl_gpu::ZBCSetTable(const std::vector<u8>& input, std::vector<
IoctlZbcSetTable params{}; IoctlZbcSetTable params{};
std::memcpy(&params, input.data(), input.size()); std::memcpy(&params, input.data(), input.size());
// TODO(ogniK): What does this even actually do? // TODO(ogniK): What does this even actually do?
// Prevent null pointer being passed as arg 1
if (output.empty()) {
LOG_WARNING(Service_NVDRV, "Avoiding passing null pointer to memcpy");
} else {
std::memcpy(output.data(), &params, output.size()); std::memcpy(output.data(), &params, output.size());
}
return NvResult::Success; return NvResult::Success;
} }