[Coverity CID: 236015] Out-of-bounds access in tests/subsys/logging/log_api/src/mock_backend.c

[zephyrbot]

Static code scan issues found in file:

https://github.com/zephyrproject-rtos/zephyr/tree/0622bde3bf22926544c1aaef4c01dfa23d79f26e/tests/subsys/logging/log_api/src/mock_backend.c#L372

Category: Memory - corruptions
Function: sync_hexdump
Component: Tests
CID: 236015

Details:

zephyr/tests/subsys/logging/log_api/src/mock_backend.c

Line 372 in 0622bde

zassert_equal(memcmp(data, exp->data, length), 0, NULL);

366      zassert_equal(src_level.source_id, exp->source_id, NULL);
367      zassert_equal(src_level.domain_id, exp->domain_id, NULL);
368     
369      zassert_equal(0, strcmp(metadata, exp->str), NULL);
370      zassert_equal(length, exp->data_len, NULL);
371      if (length > sizeof(exp->data)) {
>>>     CID 236015:  Memory - corruptions  (OVERRUN)
>>>     Overrunning array "exp->data" of 32 bytes by passing it to a function which accesses it at byte offset 32 using argument "length" (which evaluates to 33). [Note: The source code implementation of the function has been overridden by a builtin model.]
372              zassert_equal(memcmp(data, exp->data, length), 0, NULL);
373      }
374     }
375     
376     const struct log_backend_api mock_log_backend_api = {
377      .process = IS_ENABLED(CONFIG_LOG2) ? process : NULL,

Please fix or provide comments in coverity using the link:

https://scan9.coverity.com/reports.htm#v29271/p12996

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.