summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorJukka Rissanen <jukka.rissanen@linux.intel.com>2012-11-12 12:07:55 (GMT)
committerPatrik Flykt <patrik.flykt@linux.intel.com>2012-11-23 10:58:52 (GMT)
commit5fb8847d3b6454a8467c0b3e3f5a9db2115f8655 (patch)
tree40f7521407a57c110739155017f1244f59e46d4b /scripts
parent7270920bf73fe37ff15a25360dd1e63bad239515 (diff)
downloadconnman-5fb8847d3b6454a8467c0b3e3f5a9db2115f8655.zip
connman-5fb8847d3b6454a8467c0b3e3f5a9db2115f8655.tar.gz
connman-5fb8847d3b6454a8467c0b3e3f5a9db2115f8655.tar.bz2
openconnect-script: Print errors to syslog instead of stderr
Use syslog for error printing because normally the stderr goes to /dev/null and we do not see the errors.
Diffstat (limited to 'scripts')
-rw-r--r--scripts/openconnect-script.c43
1 files changed, 32 insertions, 11 deletions
diff --git a/scripts/openconnect-script.c b/scripts/openconnect-script.c
index ef81c94..d6e42eb 100644
--- a/scripts/openconnect-script.c
+++ b/scripts/openconnect-script.c
@@ -26,11 +26,22 @@
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
+#include <syslog.h>
+#include <libgen.h>
#include <dbus/dbus.h>
extern char **environ;
+static void print(const char *format, ...)
+{
+ va_list ap;
+
+ va_start(ap, format);
+ vsyslog(LOG_INFO, format, ap);
+ va_end(ap);
+}
+
static void append(DBusMessageIter *dict, const char *pattern)
{
DBusMessageIter entry;
@@ -68,6 +79,9 @@ int main(int argc, char *argv[])
DBusMessage *msg;
DBusMessageIter iter, dict;
char **envp, *busname, *reason, *interface, *path;
+ int ret = 0;
+
+ openlog(basename(argv[0]), LOG_NDELAY | LOG_PID, LOG_DAEMON);
busname = getenv("CONNMAN_BUSNAME");
interface = getenv("CONNMAN_INTERFACE");
@@ -76,31 +90,33 @@ int main(int argc, char *argv[])
reason = getenv("reason");
if (!busname || !interface || !path || !reason) {
- fprintf(stderr, "Required environment variables not set\n");
- return 1;
+ print("Required environment variables not set");
+ ret = 1;
+ goto out;
}
if (strcmp(reason, "pre-init") == 0)
- return 0;
+ goto out;
dbus_error_init(&error);
conn = dbus_bus_get(DBUS_BUS_SYSTEM, &error);
if (conn == NULL) {
if (dbus_error_is_set(&error) == TRUE) {
- fprintf(stderr, "%s\n", error.message);
+ print("%s", error.message);
dbus_error_free(&error);
} else
- fprintf(stderr, "Failed to get on system bus\n");
- return 0;
+ print("Failed to get on system bus");
+
+ goto out;
}
msg = dbus_message_new_method_call(busname, path,
interface, "notify");
if (msg == NULL) {
dbus_connection_unref(conn);
- fprintf(stderr, "Failed to allocate method call\n");
- return 0;
+ print("Failed to allocate method call");
+ goto out;
}
dbus_message_set_no_reply(msg, TRUE);
@@ -121,8 +137,10 @@ int main(int argc, char *argv[])
dbus_message_iter_close_container(&iter, &dict);
- if (dbus_connection_send(conn, msg, NULL) == FALSE)
- fprintf(stderr, "Failed to send message\n");
+ if (dbus_connection_send(conn, msg, NULL) == FALSE) {
+ print("Failed to send message");
+ goto out;
+ }
dbus_connection_flush(conn);
@@ -130,5 +148,8 @@ int main(int argc, char *argv[])
dbus_connection_unref(conn);
- return 0;
+out:
+ closelog();
+
+ return ret;
}