On Mon, May 02, 2016 at 12:37:09AM -0600, Jan Beulich wrote:
> >>> On 27.04.16 at 21:27, <konrad.w...@oracle.com> wrote:
> > @@ -506,6 +518,37 @@ static int prepare_payload(struct payload *payload,
> > }
> > }
> >
> > + sec = xsplice_elf_sec_by_name(elf, ELF_BUILD_ID_NOTE);
> > + if ( sec )
> > + {
> > + n = sec->load_addr;
> > +
> > + if ( sec->sec->sh_size <= sizeof(*n) )
> > + return -EINVAL;
> > +
> > + if ( xen_build_id_check(n, sec->sec->sh_size,
> > + &payload->id.p, &payload->id.len) )
> > + return -EINVAL;
> > +
> > + if ( !payload->id.len || !payload->id.p )
> > + return -EINVAL;
> > + }
> > +
> > + sec = xsplice_elf_sec_by_name(elf, ELF_XSPLICE_DEPENDS);
> > + {
>
> Looks like an "if ( sec )" got lost here.
<grumbles>
Patch sent. Thank you for spotting that!
>
> Jan
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
