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

[Tom Rini]

On Wed, Nov 08, 2023 at 03:24:38AM +0000, Alexander Gendin wrote:
> On Mon, Nov 06, 2023 at 03:27:52PM -0500, Tom Rini <trini at 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 at 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 at 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.

Thanks for the patch, I'll make sure to grab it before the next -rc.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231108/00e1e1fc/attachment.sig>