Messages in this thread | | | Subject | Re: [PATCH 1/1] Additional strict check on ELF file. Checks segments are followed in order of 'p_vaddr ' value ascending. It fixes erorr in total_mapping_size with computation total size. This error happens if segments in ELF file are not in order. | From | Randy Dunlap <> | Date | Mon, 26 Feb 2018 09:48:23 -0800 |
| |
On 02/26/2018 07:46 AM, Ilya Smith wrote: > Signed-off-by: Ilya Smith <blackzert@gmail.com> > --- > fs/binfmt_elf.c | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 51 insertions(+) > > diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c > index bdb201230bae..970b42044240 100644 > --- a/fs/binfmt_elf.c > +++ b/fs/binfmt_elf.c > @@ -524,6 +524,52 @@ static inline int arch_check_elf(struct elfhdr *ehdr, bool has_interp, > > #endif /* !CONFIG_ARCH_BINFMT_ELF_STATE */ > > +/** > + * elf_check_phdr() - common check ELF program header. > + * @phdr: The program header to check > + * @phdr_num: Count of program headers in @phdr from elf header. > + * > + * Checks ELF binary meets specification. > + * > + * Return: Zero to proceed with ELF load, non-zero to faile the ELF load
fail
> + * with that return code. > + */ > +static int elf_check_phdr(struct elf_phdr *phdr, unsigned long phdr_num) > +{
And it would be nicer/better to have all of the "intro" text in the body of this message instead of in another email.
thanks, -- ~Randy
| |