On 07/13/16 at 09:35pm, Lans Zhang wrote: > The commit e68503bd68 forgot to set digest_len and thus cause the following > error reported by kexec when launching a crash kernel: > "kexec_file_load failed: Bad message" > > Fixes: e68503bd68 (KEYS: Generalise system_verify_data() to provide access to internal content) > Signed-off-by: Lans Zhang <jia.zhang at windriver.com> > Cc: David Howells <dhowells at redhat.com> > Cc: Dave Young <dyoung at redhat.com> > Cc: Baoquan He <bhe at redhat.com> > Cc: Vivek Goyal <vgoyal at redhat.com> > --- > crypto/asymmetric_keys/mscode_parser.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/crypto/asymmetric_keys/mscode_parser.c b/crypto/asymmetric_keys/mscode_parser.c > index 6a76d5c..9492e1c 100644 > --- a/crypto/asymmetric_keys/mscode_parser.c > +++ b/crypto/asymmetric_keys/mscode_parser.c > @@ -124,5 +124,10 @@ int mscode_note_digest(void *context, size_t hdrlen, > struct pefile_context *ctx = context; > > ctx->digest = kmemdup(value, vlen, GFP_KERNEL); > - return ctx->digest ? 0 : -ENOMEM; > + if (!ctx->digest) > + return -ENOMEM; > + > + ctx->digest_len = vlen; > + > + return 0; > } > -- > 1.9.1 > > > _______________________________________________ > kexec mailing list > kexec at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/kexec Tested-by: Dave Young <dyoung at redhat.com> Thanks Dave
