linux-kernel - Re: + uml-make-execvp-safe-for-our-usage.patch added to -mm tree

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <20061124134621.be9835e7.akpm@osdl.org>
Date:	Fri, 24 Nov 2006 13:46:21 -0800
From:	Andrew Morton <akpm@...l.org>
To:	Jeff Dike <jdike@...toit.com>
Cc:	linux-kernel@...r.kernel.org, blaisorblade@...oo.it
Subject: Re: + uml-make-execvp-safe-for-our-usage.patch added to -mm tree

On Fri, 24 Nov 2006 14:50:11 -0500
Jeff Dike <jdike@...toit.com> wrote:

> On Tue, Oct 17, 2006 at 05:14:35PM -0700, akpm@...l.org wrote:
> > The patch titled
> > 
> >      uml: make execvp safe for our usage
> > 
> > has been added to the -mm tree.  Its filename is
> > 
> >      uml-make-execvp-safe-for-our-usage.patch
> > 
> 
> I had previously objected to this patch on grounds of taste.  Since
> this fixes a number of problems, and I don't have any solutions which
> are huge improvements, I withdraw those objections.
> 
> Feel free to send to Linus.

OK.  Is it needed for 2.6.19?

Also, I'm still sitting on the below.  I have a note that you nacked it,
but I forget why.


From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@...oo.it>

Fix prototypes in user.h - was needed when including user.h in kernelspace, as
we did in previous patch.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@...oo.it>
Cc: Jeff Dike <jdike@...toit.com>
Signed-off-by: Andrew Morton <akpm@...l.org>
---

 arch/um/include/user.h |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff -puN arch/um/include/user.h~uml-fix-prototypes arch/um/include/user.h
--- a/arch/um/include/user.h~uml-fix-prototypes
+++ a/arch/um/include/user.h
@@ -6,6 +6,10 @@
 #ifndef __USER_H__
 #define __USER_H__
 
+/* Both on kernelspace and userspace this will provide the size_t definition. It should, at
+ * least. But on userspace it won't hurt surely. */
+#include <linux/types.h>
+
 extern void panic(const char *fmt, ...)
 	__attribute__ ((format (printf, 1, 2)));
 extern int printk(const char *fmt, ...)
@@ -13,9 +17,8 @@ extern int printk(const char *fmt, ...)
 extern void schedule(void);
 extern int in_aton(char *str);
 extern int open_gdb_chan(void);
-/* These use size_t, however unsigned long is correct on both i386 and x86_64. */
-extern unsigned long strlcpy(char *, const char *, unsigned long);
-extern unsigned long strlcat(char *, const char *, unsigned long);
+extern size_t strlcpy(char *, const char *, size_t);
+extern size_t strlcat(char *, const char *, size_t);
 
 #endif
 
_

-
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/