linux-kernel - Re: [PATCH] Kcore elf note namesz field fix

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 8 Sep 2006 20:13:44 +0900
From:	Horms <horms@...ge.net.au>
To:	Vivek Goyal <vgoyal@...ibm.com>
Cc:	Fastboot mailing list <fastboot@...ts.osdl.org>,
	"Eric W. Biederman" <ebiederm@...ssion.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Kcore elf note namesz field fix

[ This is a repost, I mucked up the headers the first time around ]

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@...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@...ge.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/