On Tue, Jan 22, 2019 at 08:32:48AM +1100, matthew green wrote:
> > > @@ -472,6 +472,9 @@
> > >   const char *bootname = device_xname(bdv);
> > >   size_t len = strlen(bootname);
> > >  
> > > + if (bdv == NULL)
> > > +         return 0;
> > > +
> > 
> > This looked suspicious, even before I read the code.
> > 
> > The question is if it is ever legitimate for bdv to be NULL.
> 
> infact, bdv has already been dereferenced at this point:
> the assignment to bootname 3 lines up.

Argh.  That's what I get for first removing the code I added which
correctly moved the initialization of bootname and len to after the
bdv==NULL check and the re-adding it at midnight.

--chris

Reply via email to