linux-toradex.git/kernel/exec_domain.c, branch v3.17-rc6

kernel/exec_domain.c: code clean-up

2014-06-04T23:54:15+00:00

Fix checkpatch warnings about EXPORT_SYMBOL and return()

Signed-off-by: Fabian Frederick 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

sys_personality: remove the bogus checks in sys_personality()->__set_personality() path

2010-08-10T03:45:05+00:00

Cleanup, no functional changes.

- __set_personality() always changes ->exec_domain/personality, the
  special case when ->exec_domain remains the same buys nothing but
  complicates the code. Unify both cases to simplify the code.

- The -EINVAL check in sys_personality() was never right. If we assume
  that set_personality() can fail we should check the value it returns
  instead of verifying that task->personality was actually changed.

  Remove it. Before the previous patch it was possible to hit this case
  due to overflow problems, but this -EINVAL just indicated the kernel
  bug.

OTOH, probably it makes sense to change lookup_exec_domain() to return
ERR_PTR() instead of default_exec_domain if the search in exec_domains
list fails, and report this error to the user-space.  But this means
another user-space change, and we have in-kernel users which need fixes.
For example, PER_OSF4 falls into PER_MASK for unkown reason and nobody
cares to register this domain.

Signed-off-by: Oleg Nesterov 
Cc: Wenming Zhang 
Cc: "H. Peter Anvin" 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

sys_personality: change sys_personality() to accept "unsigned int" instead of u_long

2010-06-04T22:21:45+00:00

task_struct->pesonality is "unsigned int", but sys_personality() paths use
"unsigned long pesonality".  This means that every assignment or
comparison is not right.  In particular, if this argument does not fit
into "unsigned int" __set_personality() changes the caller's personality
and then sys_personality() returns -EINVAL.

Turn this argument into "unsigned int" and avoid overflows.  Obviously,
this is the user-visible change, we just ignore the upper bits.  But this
can't break the sane application.

There is another thing which can confuse the poorly written applications.
User-space thinks that this syscall returns int, not long.  This means
that the returned value can be negative and look like the error code.  But
note that libc won't be confused and thus errno won't be set, and with
this patch the user-space can never get -1 unless sys_personality() really
fails.  And, most importantly, the negative RET != -1 is only possible if
that app previously called personality(RET).

Pointed-out-by: Wenming Zhang 
Suggested-by: Linus Torvalds 
Signed-off-by: Oleg Nesterov 
Cc: "H. Peter Anvin" 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Get rid of indirect include of fs_struct.h

2009-04-01T03:00:27+00:00

Don't pull it in sched.h; very few files actually need it and those
can include directly.  sched.h itself only needs forward declaration
of struct fs_struct;

Signed-off-by: Al Viro

Kill unsharing fs_struct in __set_personality()

2009-04-01T03:00:25+00:00

That's a rudiment of altroot support.  I.e. it should've been buried
a long time ago.

Signed-off-by: Al Viro

[CVE-2009-0029] System call wrappers part 04

2009-01-14T13:15:19+00:00

Signed-off-by: Heiko Carstens

proc: move /proc/execdomains to kernel/exec_domain.c

2008-10-23T10:30:41+00:00

Signed-off-by: Alexey Dobriyan

[PATCH] kill altroot

2008-07-27T00:53:20+00:00

long overdue...

Signed-off-by: Al Viro

remove CONFIG_KMOD from core kernel code

2008-07-22T09:24:31+00:00

Always compile request_module when the kernel allows modules.

Signed-off-by: Johannes Berg 
Signed-off-by: Rusty Russell

whitespace fixes: execution domains

2007-10-18T21:37:26+00:00

Signed-off-by: Daniel Walker 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds