Skip to content

Commit ee749e4

Browse files
committed
Coverity: Fix 1595568, 1595570 Use of auto that causes a copy
Use `const auto &` instead of `auto` in the mock parameter struct accesses.
1 parent d08fc6a commit ee749e4

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

source/adapters/mock/ur_mock.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,14 @@ namespace driver {
1717
context_t d_context;
1818

1919
ur_result_t mock_urPlatformGetApiVersion(void *pParams) {
20-
auto params = *static_cast<ur_platform_get_api_version_params_t *>(pParams);
20+
const auto &params =
21+
*static_cast<ur_platform_get_api_version_params_t *>(pParams);
2122
**params.ppVersion = d_context.version;
2223
return UR_RESULT_SUCCESS;
2324
}
2425

2526
ur_result_t mock_urPlatformGetInfo(void *pParams) {
26-
auto params = *static_cast<ur_platform_get_info_params_t *>(pParams);
27+
const auto &params = *static_cast<ur_platform_get_info_params_t *>(pParams);
2728
if (!*params.phPlatform) {
2829
return UR_RESULT_ERROR_INVALID_NULL_HANDLE;
2930
}
@@ -49,7 +50,7 @@ ur_result_t mock_urPlatformGetInfo(void *pParams) {
4950

5051
//////////////////////////////////////////////////////////////////////////
5152
ur_result_t mock_urDeviceGetInfo(void *pParams) {
52-
auto params = *static_cast<ur_device_get_info_params_t *>(pParams);
53+
const auto &params = *static_cast<ur_device_get_info_params_t *>(pParams);
5354
switch (*params.ppropName) {
5455
case UR_DEVICE_INFO_TYPE:
5556
if (*params.ppPropValue != nullptr) {

0 commit comments

Comments
 (0)