On Fri, Jul 15, 2022 at 05:21:21PM -0400, Alec Brown wrote: > On Thu, Jul 14, 2022 at 03:38:04PM +0100, Darren Kenny wrote: > > Hi Ross, > > > > This looks good to me. > > > > On Thursday, 2022-07-14 at 09:41:28 -04, Ross Philipson wrote: > > > The way the code is written the tofree variable would never be > > > passed to the free_subchunk() function uninitialized. Coverity > > > cannot determine this and flags the situation as "Using uninitialized > > > value...". The fix is just to initialize the local struct. > > > > > > Fixes: CID 314016 > > > > > > Signed-off-by: Ross Philipson <ross.philip...@oracle.com> > > > > Reviewed-by: Darren Kenny <darren.ke...@oracle.com> > > I ran this through a private Coverity scan which marked the bug as eliminated > and didn't have any issues running it on a VM. > > Tested-by: Alec Brown <alec.r.br...@oracle.com>
Reviewed-by: Daniel Kiper <daniel.ki...@oracle.com> Daniel _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel
