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

[Tom Rini <trini@konsulko.com>]

[-- 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 --]