Fwd: New Defects reported by Coverity Scan for Das U-Boot

Tom Rini
2024-11-15T13:27:50Z

[-- Attachment #1: Type: text/plain, Size: 3031 bytes --]

Hey all,

Here's the latest report.

---------- Forwarded message ---------
From: <scan-admin@coverity.com>
Date: Thu, Nov 14, 2024, 10:40 PM
Subject: New Defects reported by Coverity Scan for Das U-Boot
To: <tom.rini@gmail.com>

Hi,

Please find the latest report on new defect(s) introduced to Das U-Boot
found with Coverity Scan.

2 new defect(s) introduced to Das U-Boot found with Coverity Scan.
3 defect(s), reported by Coverity Scan earlier, were marked fixed in the
recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 2 of 2 defect(s)

** CID 514958: Error handling issues (CHECKED_RETURN)
/test/cmd/hash.c: 80 in dm_test_cmd_hash_sha256()

________________________________________________________________________________________________________
*** CID 514958: Error handling issues (CHECKED_RETURN)
/test/cmd/hash.c: 80 in dm_test_cmd_hash_sha256()
74 ut_assertok(run_command("hash sha256 $loadaddr 0 foo; echo $foo",
0));
75 console_record_readline(uts->actual_str, sizeof(uts->actual_str));
76 ut_asserteq_ptr(uts->actual_str,
77 strstr(uts->actual_str, "sha256 for "));
78 ut_assert(strstr(uts->actual_str,
79
"e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855"));
>>> CID 514958: Error handling issues (CHECKED_RETURN)
>>> Calling "ut_check_console_line" without checking return value (as
is done elsewhere 683 out of 690 times).
80 ut_check_console_line(uts,
81
"e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855");
82
83 if (!CONFIG_IS_ENABLED(HASH_VERIFY)) {
84 ut_assert(run_command("hash -v sha256 $loadaddr 0 foo", 0));
85 ut_check_console_line(uts, "hash - compute hash message
digest");

** CID 514957: Error handling issues (CHECKED_RETURN)
/test/cmd/hash.c: 36 in dm_test_cmd_hash_md5()

________________________________________________________________________________________________________
*** CID 514957: Error handling issues (CHECKED_RETURN)
/test/cmd/hash.c: 36 in dm_test_cmd_hash_md5()
30 ut_assertok(run_command("hash md5 $loadaddr 0 foo; echo $foo", 0));
31 console_record_readline(uts->actual_str, sizeof(uts->actual_str));
32 ut_asserteq_ptr(uts->actual_str,
33 strstr(uts->actual_str, "md5 for "));
34 ut_assert(strstr(uts->actual_str,
35 "d41d8cd98f00b204e9800998ecf8427e"));
>>> CID 514957: Error handling issues (CHECKED_RETURN)
>>> Calling "ut_check_console_line" without checking return value (as
is done elsewhere 683 out of 690 times).
36 ut_check_console_line(uts, "d41d8cd98f00b204e9800998ecf8427e");
37
38 if (!CONFIG_IS_ENABLED(HASH_VERIFY)) {
39 ut_assert(run_command("hash -v sha256 $loadaddr 0 foo", 0));
40 ut_check_console_line(uts, "hash - compute hash message
digest");
41

----- End forwarded message -----

--
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]