man/systemd.netdev.xml

@@ -629,7 +629,7 @@
           endpoint answers ARP requests from the local bridge on behalf
           of remote Distributed Overlay Virtual Ethernet
           <ulink url="https://en.wikipedia.org/wiki/Distributed_Overlay_Virtual_Ethernet">
-          (DOVE)</ulink> clients. Defaults to false.</para>
+          (DVOE)</ulink> clients. Defaults to false.</para>
         </listitem>
       </varlistentry>
       <varlistentry>

src/core/job.c

@@ -99,8 +99,12 @@ Job* job_free(Job *j) {
         assert(!j->subject_list);
         assert(!j->object_list);
 
-        while (!LIST_IS_EMPTY(j->triggered_by))
-                LIST_POP(triggered_by, j->triggered_by);
+        do {
+                Unit *tu = NULL;
+
+                LIST_FOREACH(triggered_by, tu, j->triggered_by)
+                        LIST_REMOVE(triggered_by, j->triggered_by, tu);
+        } while (!LIST_IS_EMPTY(j->triggered_by));
 
         job_unlink(j);
 

src/core/service.c

@@ -1482,27 +1482,36 @@ static int service_create_monitor_md_env(Job *j, char **ret) {
                 if (!env_source)
                         continue;
 
-                /* Add the environment variable name first. */
-                if (first && !strextend(&var, "MONITOR_METADATA="))
-                        return -ENOMEM;
+                if (first) {
+                        /* Add the environment variable name first. */
+                        r = strextendf(&var, "MONITOR_METADATA=");
+                        if (r < 0)
+                                return r;
 
-                if (!strextend(&var, !first ? list_delim : "", "SERVICE_RESULT=", service_result_to_string(env_source->result)))
-                        return -ENOMEM;
+                }
+
+                r = strextendf(&var, "%sSERVICE_RESULT=%s",
+                               !first ? list_delim : "", service_result_to_string(env_source->result));
+                if (r < 0)
+                        return r;
 
                 first = false;
 
                 if (env_source->main_exec_status.pid > 0 &&
                     dual_timestamp_is_set(&env_source->main_exec_status.exit_timestamp)) {
-                        if (!strextend(&var, ",EXIT_CODE=", sigchld_code_to_string(env_source->main_exec_status.code)))
-                                return -ENOMEM;
+                        r = strextendf(&var, ",EXIT_CODE=%s",
+                                       sigchld_code_to_string(env_source->main_exec_status.code));
+                        if (r < 0)
+                                return r;
 
-                        if (env_source->main_exec_status.code == CLD_EXITED) {
+                        if (env_source->main_exec_status.code == CLD_EXITED)
                                 r = strextendf(&var, ",EXIT_STATUS=%i",
                                                env_source->main_exec_status.status);
-                                if (r < 0)
-                                        return r;
-                        } else if (!strextend(&var, ",EXIT_STATUS=", signal_to_string(env_source->main_exec_status.status)))
-                                return -ENOMEM;
+                        else
+                                r = strextendf(&var, ",EXIT_STATUS=%s",
+                                               signal_to_string(env_source->main_exec_status.status));
+                        if (r < 0)
+                                return r;
                 }
 
                 if (!sd_id128_is_null(UNIT(env_source)->invocation_id)) {
@@ -1512,8 +1521,9 @@ static int service_create_monitor_md_env(Job *j, char **ret) {
                                 return r;
                 }
 
-                if (!strextend(&var, ",UNIT=", UNIT(env_source)->id))
-                        return -ENOMEM;
+                r = strextendf(&var, ",UNIT=%s", UNIT(env_source)->id);
+                if (r < 0)
+                        return r;
         }
 
         *ret = TAKE_PTR(var);

src/shared/watchdog.c

@@ -133,10 +133,6 @@ static int open_watchdog(void) {
         fn = !watchdog_device || path_equal(watchdog_device, "/dev/watchdog") ?
                 "/dev/watchdog0" : watchdog_device;
 
-        r = free_and_strdup(&watchdog_device, fn);
-        if (r < 0)
-                return log_oom_debug();
-
         watchdog_fd = open(fn, O_WRONLY|O_CLOEXEC);
         if (watchdog_fd < 0)
                 return log_debug_errno(errno, "Failed to open watchdog device %s, ignoring: %m", fn);

src/shutdown/shutdown.c

@@ -531,9 +531,8 @@ int main(int argc, char *argv[]) {
                           need_md_detach ? " MD devices," : "");
         }
 
-        /* We're done with the watchdog. Note that the watchdog is explicitly not
-         * stopped here. It remains active to guard against any issues during the
-         * rest of the shutdown sequence. */
+        /* We're done with the watchdog. */
+        watchdog_close(true);
         watchdog_free_device();
 
         arguments[0] = NULL;