Age | Commit message (Collapse) | Author |
|
* init/init.c (S_startup_essential_task): Fix port leak.
|
|
Found using the Clang Static Analyzer.
* init/init.c (process_signal): Fix memory leak.
|
|
This is based on a fragment of Guillem Jovers patch presented here:
http://lists.gnu.org/archive/html/bug-hurd/2006-02/msg00081.html
It has been refreshed, updated and the copyright year is adjusted
properly. It has been complemented with the necessary features to
address the issues the original patch set out to address, namely
that killall5 freezes the proc translator before it tries to walk
over /proc/*/stat to decide which process to kill. Prior to this
patch (and the one marking the procfs server as important
process), killall5 would deadlock trying to walk over the proc
file system.
Ironically it would not have killed any process later on even if
it had the chance, since two values obtained from /proc/*/stat
are currently hardcoded to zero in our procfs. Patches addressing
the problem as a whole are prepared and will be sent as a follow
up.
* init/init.c (launch_core_servers): Mark init, auth, proc and fs
servers as important.
(start_child): Mark the real init as important.
(S_startup_important_task): Mark exec server as important.
|
|
* include/pids.h: New file.
* init/init.c (frob_kernel_process): Use symbolic name for kernel process.
* exec/exec.c (S_exec_init): Use symbolic name for startup process.
* libdiskfs/boot-start.c (diskfs_S_fsys_init): Likewise.
* libdiskfs/init-startup.c (_diskfs_init_completed): Likewise.
* pfinet/main.c (arrange_shutdown_notification): Likewise.
* proc/mgt.c (create_startup_proc): Likewise.
* proc/main.c (main): Use symbolic name for proc process.
|
|
* init/init.c (start_child): Don't free `arg' before it is used.
|
|
* init/init.c (launch_something): Check with file_name_lookup that runsystem
script exists before calling start_child.
* daemons/console-run.c (main): Check with file_name_lookup that runsystem
script exists before opening a console for it.
|
|
init/init.c (S_msg_sig_post_untraced): Ignore result of
msg_sig_post_untraced_reply.
(S_msg_sig_post): Likewise.
|
|
* auth/auth.c (S_auth_server_authenticate): Check result of
auth_server_authenticate_reply stub.
* init/init.c (S_msg_sig_post_untraced): Check result of
msg_sig_post_untraced_reply
(S_msg_sig_post): Check result of msg_sig_post_reply.
|
|
|
|
|
|
|
|
* init.c (main): Only pass ARGP_NO_ERRS to argp_parse when invoked
by the bootstrap filesystem.
(flags): New variable.
|
|
* init.c (reboot_system): u_int -> size_t
(reboot_system): Likewise.
(S_msg_report_wait): int -> mach_msg_id_t
|
|
* init.c (reboot_mach): Use ERR, not errno.
(run): Likewise.
(lauch_core_servers): Likewise.
(run_for_real): Use MACH_PORT_NULL explicitly.
(start_child): Likewise.
|
|
* init.c [KERN_INVALID_LEDGER]:
Pass extra arguments to task_create for OSF variant.
|
|
* init.c (main): Pass ARGP_NO_ERRS|ARGP_IN_ORDER to argp_parse.
|
|
* init.c [! SPLIT_INIT]: All this code removed.
[SPLIT_INIT]: This code no longer conditional.
(_PATH_RUNCOM, mapped_time, do_fastboot): Removed.
* Makefile (split-init): Variable removed.
(SRCS): Add stubs.c here unconditionally.
[$(split-init) = no]: Conditional defns removed.
(LCLHDRS): Variable removed.
* ttys.c, ttys.h: Files removed.
|
|
* init.c (kernel_command_line): Variable removed.
(options, parse_opt): Remove -K option.
(frob_kernel_process): Collect arguments from &global_argv[1]
instead of slicing up kernel_command_line.
(start_child): Take a second arg, if non-null more arguments for PROG.
(launch_something): Change caller. On first try, pass &global_argv[1].
|
|
* init.c (reboot_system): Don't pass null pointer to RPC out param.
Thanks to Moritz Schulte <moritz@chaosdorf.de> for finding this.
|
|
2001-01-17 Neal H Walfield <neal@cs.uml.edu>
* console-run.c (open_console): Conform to new
fshelp_start_translator semantics.
init/
2001-01-17 Neal H Walfield <neal@cs.uml.edu>
* init.c (open_console): Conform to new fshelp_start_translator
semantics.
libfshelp/
2001-01-17 Neal H Walfield <neal@cs.uml.edu>
* fshelp.h: Add two new parameters to fshelp_open_fn_t:
a port to the new task and a cookie.
Add a new parameter, cookie, to fshelp_start_translator and
fshelp_start_translator_long that will be passed to
fshelp_open_fn_t.
* fetch-root.c (fshelp_fetch_root): Conform to new
fshelp_start_translator_long semantics.
* start-translator-long.c (service_fsys_startup): Likewise.
(fshelp_start_translator_long): Likewise.
* start-translator.c (fshelp_start_translator): Likewise.
libtreefs/
Conform to new fshelp_start_translator semantics.
trans/
2001-01-17 Neal H Walfield <neal@cs.uml.edu>
* pump.c (start_pfinet): Conform to new fshelp_start_translator
semantics.
utils/
2001-01-17 Neal H Walfield <neal@cs.uml.edu>
* mount.c (do_mount): Conform to new fshelp_start_translator
semantics.
* settrans.c (main): Conform to new fshelp_start_translator
semantics therby allowing us to print the pid of the an
active translator.
|
|
* init.c (process_rc_script): If malloc fails, return ENOMEM.
Reported by Igor Khavkine <i_khavki@alcor.concordia.ca>.
|
|
* init.c (launch_core_servers): Don't use MOVE_SEND in Hurd RPCs.
|
|
* stubs.c (send_signal): Make MESSAGE auto instead of static.
Take new argument TIMEOUT, passed to mach_msg instead of zero.
(send_signal): For MACH_SEND_TIMED_OUT failure, destroy the
pseudo-received message to avoid leaking send rights.
* init.c (process_signal): Pass new argument, timeout of 500 ms.
|
|
* init.c (frob_kernel_process): Allocate the kernel's page in the
actual kernel task, not in our own. I wonder how this ever
worked. Also if we fail in that allocation, free the page we
allocated in our own space.
|
|
* init.c (notify_shutdown): Add const to argument type.
|
|
* init.c (frob_kernel_process): Use mmap instead of vm_allocate.
|
|
* init.c: Add #include <sys/mman.h> for munmap decl.
|
|
* init.c (reboot_system): Use munmap instead of vm_deallocate.
(frob_kernel_process): Likewise.
(kill_everyone): Likewise.
|
|
* init.c (open_console): Put this function inside [! SPLIT_INIT].
|
|
* init.c [SPLIT_INIT] (child_pid, child_task): New variables.
(process_signal, start_child, launch_something, launch_system): New
functions.
[! SPLIT_INIT] (system_state, shell_pid, rc_pid, launch_single_user,
process_rc_script, launch_multi_user, launch_system, kill_everyone,
kill_multi_user, process_signal): Variables and functions put inside
#ifndef SPLIT_INIT.
* stubs.c: New file, modified from ../proc/stubs.c.
* Makefile (SRCS): Remove ttys.c here.
[$(split-init) = yes] (SRCS): Add stubs.c
[$(split-init) = yes] (init-CPPFLAGS): New variable, -DSPLIT_INIT.
[$(split-init) = no] (SRCS): Add ttys.c only here.
[$(split-init) = no] (LDLIBS): Put defn (-lutil) under this test.
(split-init): New variable to turn on split-init, commented out.
|
|
* init.c (booted): New variable.
(open_console): Use that instead of system_state.
(S_startup_essential_task): Likewise.
(do_mach_notify_dead_name): Likewise.
(launch_system): New function.
(S_startup_essential_task): Call it.
* init.c (global_argv): Make static.
* init.c (startup_envz, startup_envz_len): New variables.
(run, run_for_real): Use those for child environment.
(main): Set them up from our ENVP argument.
1999-06-14 Roland McGrath <roland@baalperazim.frob.com>
* init.c (main): Barf if run in a normal Hurd environment (not boot).
* init.c (record_essential_task): Add missing return.
|
|
* init.c (kill_multi_user): Suggest `ps -AlM' in error message.
* init.c (record_essential_task): New function, broken out of
S_startup_essential_task.
(S_startup_essential_task): Call it.
(frob_kernel_process): Call it for kernel task port.
(kill_everyone): Remove special case for PID 2 (kernel task).
|
|
* init.c (run): Add const to SERVER arg, PROG local.
Don't trim directory name from PROG for argument.
|
|
Move all the /etc/ttys handling stuff into a separate source file.
* init.c (_PATH_LOGIN, WINDOW_DELAY, struct terminal,
nttys, ttyslen, setup_terminal, add_terminal,
init_ttys, free_ttys, startup_terminal, startup_ttys,
find_line, restart_terminal, shutdown_terminal, reread_ttys): Move to
* ttys.c: ... here, new file.
* ttys.h: New file, declare external functions in ttys.c.
* Makefile (SRCS): Add ttys.c.
(OBJS): Derive from $(SRCS).
(LCLHDRS): New variable, add ttys.h.
|
|
* init.c (request_dead_name): New function.
(S_startup_request_notification, S_startup_essential_task): Use it.
(run): Request dead-name notification for new task.
(main): Request dead-name notification for FSTASK.
(do_mach_notify_dead_name): If system_state is INITIAL, recognize
FSTASK, AUTHTASK, PROCTASK specifically and crash with appropriate msg.
1999-06-04 Roland McGrath <roland@baalperazim.frob.com>
* init.c (launch_core_servers): Use proc_child on the proc server task
to register it with itself as our child. Snarf its proc port and use
proc_mark_exec do remove that unsightly `f', as we do for the other
bootstrap processes.
|
|
* init.c (frob_kernel_process): If no --kernel-command-line option,
try `getenv ("MULTIBOOT_CMDLINE")' before defaulting to "(kernel)".
|
|
* init.c (CRASH_FLAGS): Macro removed.
(crash_flags): New variable replaces it.
(crash_mach, crash_system): Use crash_flags instead of CRASH_FLAGS.
(options, parse_opt): Add --crash-debug/-H, to set crash_flags to
RB_DEBUGGER.
|
|
* init.c (kernel_command_line): New variable.
(frob_kernel_process): New function.
(S_startup_essential_task): Call it after init_stdarrays.
(options): Add --kernel-command-line/-K option.
(parse_opt): Grok -K, set kernel_command_line.
|
|
* init.c (do_mach_notify_dead_name): Deallocate the extra
reference to NAME that the notification carries.
|
|
* init.c (shutdown_terminal): Revoke access to the terminal from
existing programs.
|
|
* init.c (main): Don't bogusly clear bootstrap_args.
Reported by OKUJI Yoshinori <okuji@kuicr.kyoto-u.ac.jp>.
|
|
* init.c (setup_terminal): Free LINE after done using it.
(process_rc_script): Likewise free RCARGS.
Both reported by Katusya Tanaka (wyvern@pb3.so-net.ne.jp).
|
|
New function.
(bootstrap_args):
Initialize to 0.
(main):
Parse args using argp.
Include <argp.h>.
(options, doc):
New variables.
|
|
|
|
|
|
|
|
proc_setexecdata.
(open_console): Likewise, for file_set_translator.
|
|
for loop.
|
|
error; tolerate all errors silently.
|
|
|