diff options
| author | Nalin Dahyabhai <nalin@redhat.com> | 2021-08-23 12:52:47 -0400 |
|---|---|---|
| committer | Nalin Dahyabhai <nalin@redhat.com> | 2021-08-23 17:23:17 -0400 |
| commit | 02e59c6683d645bd579d01f4cf6efb3647d366a5 (patch) | |
| tree | 8b9f83168bfcdac85572b2ea49dc711547a5aa85 | |
| parent | d1137664fe799bb9ca84ed9e4e0d01db6df77bad (diff) | |
| download | podman-02e59c6683d645bd579d01f4cf6efb3647d366a5.tar.gz podman-02e59c6683d645bd579d01f4cf6efb3647d366a5.tar.bz2 podman-02e59c6683d645bd579d01f4cf6efb3647d366a5.zip | |
utils.RunUnderSystemdScope(): always close Conn
Make sure we close our private connection to the bus, even if we're not
successful in in using it to ask systemd to move a unit's processes to a
specific control group.
[NO TESTS NEEDED]
Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
| -rw-r--r-- | utils/utils_supported.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/utils/utils_supported.go b/utils/utils_supported.go index 8668e3fba..ebc870d26 100644 --- a/utils/utils_supported.go +++ b/utils/utils_supported.go @@ -36,6 +36,7 @@ func RunUnderSystemdScope(pid int, slice string, unitName string) error { return err } } + defer conn.Close() properties = append(properties, systemdDbus.PropSlice(slice)) properties = append(properties, newProp("PIDs", []uint32{uint32(pid)})) properties = append(properties, newProp("Delegate", true)) @@ -54,7 +55,6 @@ func RunUnderSystemdScope(pid int, slice string, unitName string) error { } return err } - defer conn.Close() // Block until job is started <-ch |