Messages was changed due into adding new communication
interface to teamd and teamdctl. It should be more clear
from which communication interface messages comes from.
Signed-off-by: Jiří Župka <jzupka(a)redhat.com>
---
libteamdctl/cli_dbus.c | 24 ++++++++++++------------
libteamdctl/cli_usock.c | 35 ++++++++++-------------------------
2 files changed, 22 insertions(+), 37 deletions(-)
diff --git a/libteamdctl/cli_dbus.c b/libteamdctl/cli_dbus.c
index a658980..5718ca6 100644
--- a/libteamdctl/cli_dbus.c
+++ b/libteamdctl/cli_dbus.c
@@ -46,16 +46,16 @@ static int cli_dbus_check_error_msg(struct teamdctl *tdc, DBusMessage
*msg)
err_msg = dbus_message_get_error_name(msg);
if (!err_msg)
return 0;
- err(tdc, "Error message received: \"%s\"", err_msg);
+ err(tdc, "dbus: Error message received: \"%s\"", err_msg);
dbres = dbus_message_iter_init(msg, &args);
if (dbres == TRUE) {
if (dbus_message_iter_get_arg_type(&args) != DBUS_TYPE_STRING) {
- err(tdc, "Received argument is not string as expected.");
+ err(tdc, "dbus: Received argument is not string as expected.");
return -EINVAL;
}
dbus_message_iter_get_basic(&args, ¶m);
- err(tdc, "Error message content: \"%s\"", param);
+ err(tdc, "dbus: Error message content: \"%s\"", param);
}
return -EINVAL;
}
@@ -74,7 +74,7 @@ static int cli_dbus_get_reply_str(struct teamdctl *tdc, char **p_reply,
}
if (dbus_message_iter_get_arg_type(&args) != DBUS_TYPE_STRING) {
- err(tdc, "Received argument is not string as expected.");
+ err(tdc, "dbus: Received argument is not string as expected.");
return -EINVAL;
}
dbus_message_iter_get_basic(&args, ¶m);
@@ -95,12 +95,12 @@ static int cli_dbus_method_call(struct teamdctl *tdc, const char
*method_name,
char *reply;
int err;
- dbg(tdc, "Calling method \"%s\"", method_name);
+ dbg(tdc, "dbus: Calling method \"%s\"", method_name);
msg = dbus_message_new_method_call(cli_dbus->service_name,
TEAMD_DBUS_PATH, TEAMD_DBUS_IFACE,
method_name);
if (!msg) {
- err(tdc, "Failed to create message.");
+ err(tdc, "dbus: Failed to create message.");
return -ENOMEM;
}
dbus_message_iter_init_append(msg, &iter);
@@ -112,13 +112,13 @@ static int cli_dbus_method_call(struct teamdctl *tdc, const char
*method_name,
DBUS_TYPE_STRING,
&str);
if (dbres == FALSE) {
- err(tdc, "Failed to construct message.");
+ err(tdc, "dbus: Failed to construct message.");
err = -ENOMEM;
goto free_msg;
}
break;
default:
- err(tdc, "Unknown argument type requested.");
+ err(tdc, "dbus: Unknown argument type requested.");
err = -EINVAL;
goto free_msg;
}
@@ -127,12 +127,12 @@ static int cli_dbus_method_call(struct teamdctl *tdc, const char
*method_name,
dbres = dbus_connection_send_with_reply(cli_dbus->conn, msg,
&pending, TEAMDCTL_REPLY_TIMEOUT);
if (dbres == FALSE) {
- err(tdc, "Send with reply failed.");
+ err(tdc, "dbus: Send with reply failed.");
err = -ENOMEM;
goto free_msg;
}
if (!pending) {
- err(tdc, "Pending call not created.");
+ err(tdc, "dbus: Pending call not created.");
err = -ENOMEM;
goto free_msg;
}
@@ -143,7 +143,7 @@ static int cli_dbus_method_call(struct teamdctl *tdc, const char
*method_name,
msg = dbus_pending_call_steal_reply(pending);
dbus_pending_call_unref(pending);
if (!msg) {
- err(tdc, "Failed to get reply.");
+ err(tdc, "dbus: Failed to get reply.");
err = -EINVAL;
goto out;
}
@@ -186,7 +186,7 @@ static int cli_dbus_init(struct teamdctl *tdc, const char *team_name,
void *priv
dbus_error_init(&error);
cli_dbus->conn = dbus_bus_get(DBUS_BUS_SYSTEM, &error);
if (!cli_dbus->conn) {
- err(tdc, "Could not acquire the system bus: %s - %s",
+ err(tdc, "dbus: Could not acquire the system bus: %s - %s",
error.name, error.message);
err = -EINVAL;
goto free_service_name;
diff --git a/libteamdctl/cli_usock.c b/libteamdctl/cli_usock.c
index a719e11..509bbe5 100644
--- a/libteamdctl/cli_usock.c
+++ b/libteamdctl/cli_usock.c
@@ -20,11 +20,11 @@
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
-#include <stdarg.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <unistd.h>
#include <teamdctl.h>
+#include <private/misc.h>
#include "teamdctl_private.h"
#include "../teamd/teamd_usock_common.h"
@@ -52,16 +52,16 @@ static int cli_usock_process_msg(struct teamdctl *tdc, char *msg,
err(tdc, "usock: Incomplete message.\n");
return -EINVAL;;
}
- err(tdc, "Error message received: \"%s\"", str);
+ err(tdc, "usock: Error message received: \"%s\"", str);
str = teamd_usock_msg_getline(&rest);
if (!str) {
err(tdc, "usock: Incomplete message.\n");
return -EINVAL;;
}
- err(tdc, "Error message content: \"%s\"", str);
+ err(tdc, "usock: Error message content: \"%s\"", str);
return -EINVAL;;
} else {
- err(tdc, "Unsupported message type.\n");
+ err(tdc, "usock: Unsupported message type.\n");
return -EINVAL;
}
return 0;
@@ -99,22 +99,6 @@ static int cli_usock_wait_recv(int sock)
return 0;
}
-static int myasprintf(char **p_str, const char *fmt, ...)
-{
- char *newstr;
- va_list ap;
- int ret;
-
- va_start(ap, fmt);
- ret = vasprintf(&newstr, fmt, ap);
- va_end(ap);
- if (ret == -1)
- return -ENOMEM;
- free(*p_str);
- *p_str = newstr;
- return 0;
-}
-
static int cli_usock_method_call(struct teamdctl *tdc, const char *method_name,
char **p_reply, void *priv,
const char *fmt, va_list ap)
@@ -126,7 +110,7 @@ static int cli_usock_method_call(struct teamdctl *tdc, const char
*method_name,
char *replystr;
int err;
- dbg(tdc, "Calling method \"%s\"", method_name);
+ dbg(tdc, "usock: Calling method \"%s\"", method_name);
err= myasprintf(&msg, "%s\n%s\n", TEAMD_USOCK_REQUEST_PREFIX,
method_name);
if (err)
@@ -140,7 +124,7 @@ static int cli_usock_method_call(struct teamdctl *tdc, const char
*method_name,
goto free_msg;
break;
default:
- err(tdc, "Unknown argument type requested.");
+ err(tdc, "usock: Unknown argument type requested.");
err = -EINVAL;
goto free_msg;
}
@@ -153,7 +137,7 @@ static int cli_usock_method_call(struct teamdctl *tdc, const char
*method_name,
err = cli_usock_wait_recv(cli_usock->sock);
if (err) {
if (err == -ETIMEDOUT)
- dbg(tdc, "Wait for reply timed-out.");
+ dbg(tdc, "usock: Wait for reply timed-out.");
goto free_msg;
}
@@ -195,14 +179,15 @@ static int cli_usock_init(struct teamdctl *tdc, const char
*team_name,
cli_usock->sock = socket(AF_UNIX, SOCK_SEQPACKET, 0);
if (cli_usock->sock == -1) {
- err(tdc, "Failed to create socket.");
+ err(tdc, "usock: Failed to create socket.");
return -errno;
}
err = connect(cli_usock->sock, (struct sockaddr *) &addr,
strlen(addr.sun_path) + sizeof(addr.sun_family));
if (err == -1) {
- err(tdc, "Failed to connect socket (%s).", addr.sun_path);
+ err(tdc, "usock: Failed to connect socket (%s).",
+ addr.sun_path);
close(cli_usock->sock);
return -errno;
}
--
1.8.3.1