lkml.org 
[lkml]   [2006]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
Subject: Re: [PATCH] Kcore elf note namesz field fix
In-Reply-To: <20060905193222.GA29478@in.ibm.com>
X-Newsgroups: gmane.linux.kernel,gmane.comp.boot-loaders.fastboot.general

On Tue, 5 Sep 2006 15:32:22 -0400, Vivek Goyal wrote:
>
>
>
> o As per ELF specifications, it looks like that elf note "namesz" field contains
> the length of "name" including the size of null character. And
> currently we are filling "namesz" without taking into the consideration
> the null character size.
>
> o Kexec-tools performs this check deligently hence I ran into the issue
> while trying to open /proc/kcore in kexec-tools for some info.
>
> Signed-off-by: Vivek Goyal <vgoyal@in.ibm.com>

That is in keeping with my reading of pages 2-4 and 2-5
of version 1.1 of the specification (is there a later version?)

Acked: Simon Horman <horms@verge.net.au>

> ---
>
> fs/proc/kcore.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff -puN fs/proc/kcore.c~kcore-elf-note-namesz-fix fs/proc/kcore.c
> --- linux-2.6.18-rc3-1M/fs/proc/kcore.c~kcore-elf-note-namesz-fix 2006-08-31 16:10:41.000000000 -0400
> +++ linux-2.6.18-rc3-1M-root/fs/proc/kcore.c 2006-08-31 16:10:41.000000000 -0400
> @@ -100,7 +100,7 @@ static int notesize(struct memelfnote *e
> int sz;
>
> sz = sizeof(struct elf_note);
> - sz += roundup(strlen(en->name), 4);
> + sz += roundup((strlen(en->name) + 1), 4);
> sz += roundup(en->datasz, 4);
>
> return sz;
> @@ -116,7 +116,7 @@ static char *storenote(struct memelfnote
>
> #define DUMP_WRITE(addr,nr) do { memcpy(bufp,addr,nr); bufp += nr; } while(0)
>
> - en.n_namesz = strlen(men->name);
> + en.n_namesz = strlen(men->name) + 1;
> en.n_descsz = men->datasz;
> en.n_type = men->type;
>
> _

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-09-08 11:29    [W:0.016 / U:1.252 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site