umask is not inherited to exec'ed image

[zooey]

The umask of the calling process is not inherited to any image loaded via any of the exec...() functions. According to the POSIX base specs version 7, it should be. This causes problems for the git test suite and (I suppose) for the mercurial test suite, too.

[zooey]

a tiny test exposing the problem - just set the umask to some specific value and, invoking the test, you'll see that it will have been reverted to '022'.

[bonefish]

Implementation hint:

• Add a parameter for the umask to _kern_exec() and pass the current umask where invoked.
• In team.cpp pass the umask on (_user_exec() -> team_exec()) and add it to the team_arg structure (defined in same file).
• In user_runtime.h add a umask field to the user_space_program_args structure and set it in team_create_thread_start() (from the team_arg field).
• In libroot's initialize_before() set the umask from the newly introduced __gRuntimeLoader->program_args member.

[zooey]

Fixed in hrev40071 - basically followed Ingo's implementation hint (thanks!), additionally chose to not inherit the umask for images loaded by other means (load_image...).

[axeld]

Which I definitely agree with. An alternative would have been to maintain the umask kernel side, and have a syscall to retrieve it - but this way, a syscall is saved whenever it's used in userland, so it's probably the better solution. Thanks!