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