|
 |
1ff636 |
From 4e78db7126779a9f6432ead9a73cdab1087405bc Mon Sep 17 00:00:00 2001
|
|
|
1ff636 |
From: Lukas Nykryn <lnykryn@redhat.com>
|
|
|
1ff636 |
Date: Fri, 17 Apr 2015 15:12:00 +0200
|
|
|
1ff636 |
Subject: [PATCH] machine: do not rely on asprintf setting arg on error
|
|
|
1ff636 |
|
|
|
1ff636 |
Strictly speaking, the output variable is undefined if asprintf fails.
|
|
|
1ff636 |
We use the return value not the arg everywhere, and should we do here.
|
|
|
1ff636 |
|
|
|
1ff636 |
(Based on 2c07315225bef6be4830bce25a74da7f0ba4fcdc)
|
|
|
1ff636 |
---
|
|
|
1ff636 |
src/machine/machine-dbus.c | 10 ++++++----
|
|
|
1ff636 |
1 file changed, 6 insertions(+), 4 deletions(-)
|
|
|
1ff636 |
|
|
|
1ff636 |
diff --git a/src/machine/machine-dbus.c b/src/machine/machine-dbus.c
|
|
|
181b3f |
index b0f0f66e0..624a99f4e 100644
|
|
|
1ff636 |
--- a/src/machine/machine-dbus.c
|
|
|
1ff636 |
+++ b/src/machine/machine-dbus.c
|
|
|
1ff636 |
@@ -438,6 +438,7 @@ int bus_machine_method_open_login(sd_bus *bus, sd_bus_message *message, void *us
|
|
|
1ff636 |
_cleanup_close_ int master = -1;
|
|
|
1ff636 |
Machine *m = userdata;
|
|
|
1ff636 |
const char *p;
|
|
|
1ff636 |
+ char *address;
|
|
|
1ff636 |
int r;
|
|
|
1ff636 |
|
|
|
1ff636 |
if (m->class != MACHINE_CONTAINER)
|
|
|
1ff636 |
@@ -475,13 +476,14 @@ int bus_machine_method_open_login(sd_bus *bus, sd_bus_message *message, void *us
|
|
|
1ff636 |
return r;
|
|
|
1ff636 |
|
|
|
1ff636 |
#ifdef ENABLE_KDBUS
|
|
|
1ff636 |
- asprintf(&container_bus->address, "x-machine-kernel:pid=" PID_FMT ";x-machine-unix:pid=" PID_FMT, m->leader, m->leader);
|
|
|
1ff636 |
+# define ADDRESS_FMT "x-machine-kernel:pid=%1$" PID_PRI ";x-machine-unix:pid=%1$" PID_PRI
|
|
|
1ff636 |
#else
|
|
|
1ff636 |
- asprintf(&container_bus->address, "x-machine-unix:pid=" PID_FMT, m->leader);
|
|
|
1ff636 |
+# define ADDRESS_FMT "x-machine-unix:pid=%1$" PID_PRI
|
|
|
1ff636 |
#endif
|
|
|
1ff636 |
- if (!container_bus->address)
|
|
|
1ff636 |
- return -ENOMEM;
|
|
|
1ff636 |
+ if (asprintf(&address, ADDRESS_FMT, m->leader) < 0)
|
|
|
1ff636 |
+ return log_oom();
|
|
|
1ff636 |
|
|
|
1ff636 |
+ container_bus->address = address;
|
|
|
1ff636 |
container_bus->bus_client = true;
|
|
|
1ff636 |
container_bus->trusted = false;
|
|
|
1ff636 |
container_bus->is_system = true;
|