Re: [tom.rini@gmail.com: Fwd: New Defects reported by Coverity Scan for Das U-Boot]

[Alexander Gendin <agendin@matrox.com>]

On Mon, Nov 06, 2023 at 03:27:52PM -0500, Tom Rini <trini@konsulko.com> wrote:

> Hey all,
> 
> Here's the latest report. I _think_ I passed the right options to
> get_maintainer.pl such that it would only look far enough back in git to
> find the likely authors (along with listed maintainers of the files).
> 
> ---------- Forwarded message ---------
> From: <scan-admin@coverity.com>
> Date: Mon, Nov 6, 2023 at 2:58 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.
> 
> 13 new defect(s) introduced to Das U-Boot found with Coverity Scan.
> 5 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 13 of 13 defect(s)
> 
> <...skipped...>
> *** CID 467404:  Control flow issues  (DEADCODE)
> /test/cmd/mbr.c: 217 in build_mbr_parts()
> 211                                             return 1;
> 212                                     strcat(cur_buf, mbr_parts_p5);
> 213                                     bytes_remaining -= cur_str_size;
> 214
> 215                             }
> 216                             else if (num_parts > 5)
> >>>     CID 467404:  Control flow issues  (DEADCODE)
> >>>     Execution cannot reach this statement: "return 1U;".
> 217                                     return 1;
> 218                             }
> 219                     }
> 220             }
> 221
> 222             cur_str_size = sizeof(mbr_parts_tail);
> <...skipped...>
> 
> -- 
> Tom

Hi Tom,

Thanks for the report.

The patch to fix CID 467404 has been sent to the mailing list.

Regards,
Alex

--