]> granicus.if.org Git - sudo/commitdiff
Replace sudo_fatal(NULL) with an "unable to allocate memory" message
authorTodd C. Miller <Todd.Miller@sudo.ws>
Fri, 12 Oct 2018 14:39:12 +0000 (08:39 -0600)
committerTodd C. Miller <Todd.Miller@sudo.ws>
Fri, 12 Oct 2018 14:39:12 +0000 (08:39 -0600)
that includes the function name.

plugins/sudoers/sudoreplay.c
src/exec_monitor.c
src/exec_nopty.c
src/exec_pty.c

index 9ca08ececdb291159e1d16ce4da6c96be60c9db6..f7f14f1beef106d7e307b4261331ee81fefa7832 100644 (file)
@@ -571,10 +571,10 @@ xterm_get_size(int *new_rows, int *new_cols)
     /* Setup an event for reading the terminal size */
     evbase = sudo_ev_base_alloc();
     if (evbase == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     gc.ev = sudo_ev_alloc(ttyfd, SUDO_EV_READ, getsize_cb, &gc);
     if (gc.ev == NULL)
-        sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Read back terminal size response */
     if (sudo_ev_add(evbase, gc.ev, &gc.timeout, false) == -1)
index b0a362f50db9082f73e31ca584deb6d5ce5009a6..e09f2212c214bf76515e4b65f9c6328c5a8338f4 100644 (file)
@@ -457,13 +457,13 @@ fill_exec_closure_monitor(struct monitor_closure *mc,
     /* Setup event base and events. */
     mc->evbase = sudo_ev_base_alloc();
     if (mc->evbase == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Event for command status via errfd. */
     mc->errpipe_event = sudo_ev_alloc(errfd,
        SUDO_EV_READ|SUDO_EV_PERSIST, mon_errpipe_cb, mc);
     if (mc->errpipe_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->errpipe_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
@@ -471,7 +471,7 @@ fill_exec_closure_monitor(struct monitor_closure *mc,
     mc->backchannel_event = sudo_ev_alloc(backchannel,
        SUDO_EV_READ|SUDO_EV_PERSIST, mon_backchannel_cb, mc);
     if (mc->backchannel_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->backchannel_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
@@ -479,56 +479,56 @@ fill_exec_closure_monitor(struct monitor_closure *mc,
     mc->sigint_event = sudo_ev_alloc(SIGINT,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigint_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigint_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigquit_event = sudo_ev_alloc(SIGQUIT,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigquit_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigquit_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigtstp_event = sudo_ev_alloc(SIGTSTP,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigtstp_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigtstp_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigterm_event = sudo_ev_alloc(SIGTERM,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigterm_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigterm_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sighup_event = sudo_ev_alloc(SIGHUP,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sighup_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sighup_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigusr1_event = sudo_ev_alloc(SIGUSR1,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigusr1_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigusr1_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigusr2_event = sudo_ev_alloc(SIGUSR2,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigusr2_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigusr2_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     mc->sigchld_event = sudo_ev_alloc(SIGCHLD,
        SUDO_EV_SIGINFO, mon_signal_cb, mc);
     if (mc->sigchld_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(mc->evbase, mc->sigchld_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
index 3ba34fdb663e731fd8ebba819c8eccab4fbcde81..e7941cb9d808e5879dd04a83198183e00fa47b3d 100644 (file)
@@ -204,13 +204,13 @@ fill_exec_closure_nopty(struct exec_closure_nopty *ec,
     /* Setup event base and events. */
     ec->evbase = sudo_ev_base_alloc();
     if (ec->evbase == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Event for command status via errfd. */
     ec->errpipe_event = sudo_ev_alloc(errfd,
        SUDO_EV_READ|SUDO_EV_PERSIST, errpipe_cb, ec);
     if (ec->errpipe_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->errpipe_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
     sudo_debug_printf(SUDO_DEBUG_INFO, "error pipe fd %d\n", errfd);
@@ -219,77 +219,77 @@ fill_exec_closure_nopty(struct exec_closure_nopty *ec,
     ec->sigint_event = sudo_ev_alloc(SIGINT,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigint_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigint_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigquit_event = sudo_ev_alloc(SIGQUIT,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigquit_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigquit_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigtstp_event = sudo_ev_alloc(SIGTSTP,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigtstp_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigtstp_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigterm_event = sudo_ev_alloc(SIGTERM,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigterm_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigterm_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sighup_event = sudo_ev_alloc(SIGHUP,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sighup_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sighup_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigalrm_event = sudo_ev_alloc(SIGALRM,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigalrm_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigalrm_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigpipe_event = sudo_ev_alloc(SIGPIPE,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigpipe_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigpipe_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigusr1_event = sudo_ev_alloc(SIGUSR1,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigusr1_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigusr1_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigusr2_event = sudo_ev_alloc(SIGUSR2,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigusr2_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigusr2_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigchld_event = sudo_ev_alloc(SIGCHLD,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigchld_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigchld_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigcont_event = sudo_ev_alloc(SIGCONT,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->sigcont_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigcont_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
@@ -297,7 +297,7 @@ fill_exec_closure_nopty(struct exec_closure_nopty *ec,
     ec->siginfo_event = sudo_ev_alloc(SIGINFO,
        SUDO_EV_SIGINFO, signal_cb_nopty, ec);
     if (ec->siginfo_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->siginfo_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 #endif
index 4c26479ec57b06d7fae270f94740f2364f2bc428..d2088cfc4214496c52a1e86865d3c0f59199d954 100644 (file)
@@ -1149,13 +1149,13 @@ fill_exec_closure_pty(struct exec_closure_pty *ec, struct command_status *cstat,
     /* Setup event base and events. */
     ec->evbase = sudo_ev_base_alloc();
     if (ec->evbase == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Event for command status via backchannel. */
     ec->backchannel_event = sudo_ev_alloc(backchannel,
        SUDO_EV_READ|SUDO_EV_PERSIST, backchannel_cb, ec);
     if (ec->backchannel_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->backchannel_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
     sudo_debug_printf(SUDO_DEBUG_INFO, "backchannel fd %d\n", backchannel);
@@ -1164,70 +1164,70 @@ fill_exec_closure_pty(struct exec_closure_pty *ec, struct command_status *cstat,
     ec->sigint_event = sudo_ev_alloc(SIGINT,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigint_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigint_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigquit_event = sudo_ev_alloc(SIGQUIT,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigquit_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigquit_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigtstp_event = sudo_ev_alloc(SIGTSTP,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigtstp_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigtstp_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigterm_event = sudo_ev_alloc(SIGTERM,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigterm_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigterm_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sighup_event = sudo_ev_alloc(SIGHUP,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sighup_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sighup_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigalrm_event = sudo_ev_alloc(SIGALRM,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigalrm_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigalrm_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigusr1_event = sudo_ev_alloc(SIGUSR1,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigusr1_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigusr1_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigusr2_event = sudo_ev_alloc(SIGUSR2,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigusr2_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigusr2_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigchld_event = sudo_ev_alloc(SIGCHLD,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigchld_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigchld_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
     ec->sigwinch_event = sudo_ev_alloc(SIGWINCH,
        SUDO_EV_SIGINFO, signal_cb_pty, ec);
     if (ec->sigwinch_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
     if (sudo_ev_add(ec->evbase, ec->sigwinch_event, NULL, false) == -1)
        sudo_fatal(U_("unable to add event to queue"));
 
@@ -1235,7 +1235,7 @@ fill_exec_closure_pty(struct exec_closure_pty *ec, struct command_status *cstat,
     ec->fwdchannel_event = sudo_ev_alloc(backchannel,
        SUDO_EV_WRITE, fwdchannel_cb, ec);
     if (ec->fwdchannel_event == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Set the default event base. */
     sudo_ev_base_setdef(ec->evbase);
@@ -1653,7 +1653,7 @@ del_io_events(bool nonblocking)
     /* Create temporary event base for flushing. */
     evbase = sudo_ev_base_alloc();
     if (evbase == NULL)
-       sudo_fatal(NULL);
+       sudo_fatalx(U_("%s: %s"), __func__, U_("unable to allocate memory"));
 
     /* Avoid reading from /dev/tty, just flush existing data. */
     SLIST_FOREACH(iob, &iobufs, entries) {