From 73b910a341c8903cb9dda334cc39cf14e46e6106 Mon Sep 17 00:00:00 2001 From: 0x0916 Date: Mon, 5 Jun 2017 14:31:03 +0800 Subject: [PATCH] API: expose function lxc_log_init Signed-off-by: 0x0916 --- src/lxc/log.c | 41 +++++++++++++++++----------------- src/lxc/log.h | 4 ---- src/lxc/lxc_monitord.c | 10 ++++++++- src/lxc/lxccontainer.h | 16 +++++++++++++ src/lxc/tools/lxc_attach.c | 10 +++++++-- src/lxc/tools/lxc_autostart.c | 11 +++++++-- src/lxc/tools/lxc_cgroup.c | 11 +++++++-- src/lxc/tools/lxc_checkpoint.c | 11 +++++++-- src/lxc/tools/lxc_console.c | 11 +++++++-- src/lxc/tools/lxc_copy.c | 11 +++++++-- src/lxc/tools/lxc_create.c | 11 +++++++-- src/lxc/tools/lxc_destroy.c | 11 +++++++-- src/lxc/tools/lxc_device.c | 11 +++++++-- src/lxc/tools/lxc_execute.c | 12 ++++++++-- src/lxc/tools/lxc_freeze.c | 11 +++++++-- src/lxc/tools/lxc_info.c | 11 +++++++-- src/lxc/tools/lxc_init.c | 14 +++++++++--- src/lxc/tools/lxc_ls.c | 11 +++++++-- src/lxc/tools/lxc_monitor.c | 12 ++++++++-- src/lxc/tools/lxc_snapshot.c | 11 +++++++-- src/lxc/tools/lxc_start.c | 11 +++++++-- src/lxc/tools/lxc_stop.c | 11 +++++++-- src/lxc/tools/lxc_unfreeze.c | 12 ++++++++-- src/lxc/tools/lxc_wait.c | 11 +++++++-- 24 files changed, 229 insertions(+), 67 deletions(-) diff --git a/src/lxc/log.c b/src/lxc/log.c index 339b81c61..4e7ba2da9 100644 --- a/src/lxc/log.c +++ b/src/lxc/log.c @@ -44,6 +44,7 @@ #include "log.h" #include "caps.h" #include "utils.h" +#include "lxccontainer.h" /* We're logging in seconds and nanoseconds. Assuming that the underlying * datatype is currently at maximum a 64bit integer, we have a date string that @@ -543,9 +544,7 @@ extern void lxc_log_enable_syslog(void) * Called from lxc front-end programs (like lxc-create, lxc-start) to * initalize the log defaults. */ -extern int lxc_log_init(const char *name, const char *file, - const char *priority, const char *prefix, int quiet, - const char *lxcpath) +extern int lxc_log_init(struct lxc_log *log) { int lxc_priority = LXC_LOG_PRIORITY_ERROR; int ret; @@ -555,8 +554,8 @@ extern int lxc_log_init(const char *name, const char *file, return 0; } - if (priority) - lxc_priority = lxc_log_priority_to_int(priority); + if (log->priority) + lxc_priority = lxc_log_priority_to_int(log->priority); if (!lxc_loglevel_specified) { lxc_log_category_lxc.priority = lxc_priority; @@ -564,49 +563,49 @@ extern int lxc_log_init(const char *name, const char *file, } if (!lxc_quiet_specified) { - if (!quiet) + if (!log->quiet) lxc_log_category_lxc.appender->next = &log_appender_stderr; } - if (prefix) - lxc_log_set_prefix(prefix); + if (log->prefix) + lxc_log_set_prefix(log->prefix); - if (name) - log_vmname = strdup(name); + if (log->name) + log_vmname = strdup(log->name); - if (file) { - if (strcmp(file, "none") == 0) + if (log->file) { + if (strcmp(log->file, "none") == 0) return 0; - ret = __lxc_log_set_file(file, 1); + ret = __lxc_log_set_file(log->file, 1); lxc_log_use_global_fd = 1; } else { /* if no name was specified, there nothing to do */ - if (!name) + if (!log->name) return 0; ret = -1; - if (!lxcpath) - lxcpath = LOGPATH; + if (!log->lxcpath) + log->lxcpath = LOGPATH; /* try LOGPATH if lxcpath is the default for the privileged containers */ - if (!geteuid() && strcmp(LXCPATH, lxcpath) == 0) - ret = _lxc_log_set_file(name, NULL, 0); + if (!geteuid() && strcmp(LXCPATH, log->lxcpath) == 0) + ret = _lxc_log_set_file(log->name, NULL, 0); /* try in lxcpath */ if (ret < 0) - ret = _lxc_log_set_file(name, lxcpath, 1); + ret = _lxc_log_set_file(log->name, log->lxcpath, 1); /* try LOGPATH in case its writable by the caller */ if (ret < 0) - ret = _lxc_log_set_file(name, NULL, 0); + ret = _lxc_log_set_file(log->name, NULL, 0); } /* * If !file, that is, if the user did not request this logpath, then * ignore failures and continue logging to console */ - if (!file && ret != 0) { + if (!log->file && ret != 0) { INFO("Ignoring failure to open default logfile."); ret = 0; } diff --git a/src/lxc/log.h b/src/lxc/log.h index b9ee49756..8a68c8506 100644 --- a/src/lxc/log.h +++ b/src/lxc/log.h @@ -348,10 +348,6 @@ ATTR_UNUSED static inline void LXC_##PRIORITY(struct lxc_log_locinfo* locinfo, \ extern int lxc_log_fd; -extern int lxc_log_init(const char *name, const char *file, - const char *priority, const char *prefix, int quiet, - const char *lxcpath); - extern int lxc_log_set_file(int *fd, const char *fname); extern int lxc_log_syslog(int facility); extern void lxc_log_enable_syslog(void); diff --git a/src/lxc/lxc_monitord.c b/src/lxc/lxc_monitord.c index 62e212112..0f153ea40 100644 --- a/src/lxc/lxc_monitord.c +++ b/src/lxc/lxc_monitord.c @@ -44,6 +44,7 @@ #include "mainloop.h" #include "monitor.h" #include "utils.h" +#include "lxccontainer.h" #define CLIENTFDS_CHUNK 64 @@ -350,6 +351,7 @@ int main(int argc, char *argv[]) char *lxcpath = argv[1]; bool mainloop_opened = false; bool monitord_created = false; + struct lxc_log log; if (argc != 3) { fprintf(stderr, @@ -364,7 +366,13 @@ int main(int argc, char *argv[]) if (ret < 0 || ret >= sizeof(logpath)) exit(EXIT_FAILURE); - ret = lxc_log_init(NULL, logpath, "DEBUG", "lxc-monitord", 0, lxcpath); + log.name = NULL; + log.file = logpath; + log.priority = "DEBUG"; + log.prefix = "lxc-monitord"; + log.quiet = 0; + log.lxcpath = lxcpath; + ret = lxc_log_init(&log); if (ret) INFO("Failed to open log file %s, log will be lost.", lxcpath); lxc_log_options_no_override(); diff --git a/src/lxc/lxccontainer.h b/src/lxc/lxccontainer.h index 57301e658..3b45d75b7 100644 --- a/src/lxc/lxccontainer.h +++ b/src/lxc/lxccontainer.h @@ -1017,6 +1017,22 @@ int list_active_containers(const char *lxcpath, char ***names, struct lxc_contai */ int list_all_containers(const char *lxcpath, char ***names, struct lxc_container ***cret); +struct lxc_log { + const char *name; + const char *lxcpath; + const char *file; + const char *priority; + const char *prefix; + bool quiet; +}; + +/*! + *\brief Initialize the log + * + *\param log lxc log configuration. + */ +int lxc_log_init(struct lxc_log *log); + /*! * \brief Close log file. */ diff --git a/src/lxc/tools/lxc_attach.c b/src/lxc/tools/lxc_attach.c index d35ec88c1..d619e1844 100644 --- a/src/lxc/tools/lxc_attach.c +++ b/src/lxc/tools/lxc_attach.c @@ -371,6 +371,7 @@ int main(int argc, char *argv[]) { int ret = -1, r; int wexit = 0; + struct lxc_log log; pid_t pid; lxc_attach_options_t attach_options = LXC_ATTACH_OPTIONS_DEFAULT; lxc_attach_command_t command = (lxc_attach_command_t){.program = NULL}; @@ -386,8 +387,13 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - r = lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0]); + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + r = lxc_log_init(&log); if (r) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_autostart.c b/src/lxc/tools/lxc_autostart.c index a24dd7cdd..68d10144b 100644 --- a/src/lxc/tools/lxc_autostart.c +++ b/src/lxc/tools/lxc_autostart.c @@ -352,12 +352,19 @@ int main(int argc, char *argv[]) struct lxc_container **containers = NULL; struct lxc_list **c_groups_lists = NULL; struct lxc_list *cmd_group; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_cgroup.c b/src/lxc/tools/lxc_cgroup.c index 402763073..a7237a833 100644 --- a/src/lxc/tools/lxc_cgroup.c +++ b/src/lxc/tools/lxc_cgroup.c @@ -67,6 +67,7 @@ int main(int argc, char *argv[]) { char *state_object = NULL, *value = NULL; struct lxc_container *c; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); @@ -74,8 +75,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_checkpoint.c b/src/lxc/tools/lxc_checkpoint.c index c42cecdb3..0a1869009 100644 --- a/src/lxc/tools/lxc_checkpoint.c +++ b/src/lxc/tools/lxc_checkpoint.c @@ -234,6 +234,7 @@ static bool restore(struct lxc_container *c) int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; bool ret; if (lxc_arguments_parse(&my_args, argc, argv)) @@ -242,8 +243,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_console.c b/src/lxc/tools/lxc_console.c index 88a24bec3..eb57e489a 100644 --- a/src/lxc/tools/lxc_console.c +++ b/src/lxc/tools/lxc_console.c @@ -98,6 +98,7 @@ int main(int argc, char *argv[]) { int ret; struct lxc_container *c; + struct lxc_log log; ret = lxc_arguments_parse(&my_args, argc, argv); if (ret) @@ -106,8 +107,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - ret = lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0]); + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + ret = lxc_log_init(&log); if (ret) return EXIT_FAILURE; lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_copy.c b/src/lxc/tools/lxc_copy.c index 0627a6272..17d748d38 100644 --- a/src/lxc/tools/lxc_copy.c +++ b/src/lxc/tools/lxc_copy.c @@ -169,6 +169,7 @@ static int parse_ovl_mnt(char *mntstring, enum mnttype type); int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; int flags = 0; int ret = EXIT_FAILURE; @@ -178,8 +179,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(ret); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_create.c b/src/lxc/tools/lxc_create.c index b8795c446..c8691a3de 100644 --- a/src/lxc/tools/lxc_create.c +++ b/src/lxc/tools/lxc_create.c @@ -208,6 +208,7 @@ int main(int argc, char *argv[]) { struct lxc_container *c; struct bdev_specs spec; + struct lxc_log log; int flags = 0; if (lxc_arguments_parse(&my_args, argc, argv)) @@ -216,8 +217,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_destroy.c b/src/lxc/tools/lxc_destroy.c index 3f46415df..7c1c17eba 100644 --- a/src/lxc/tools/lxc_destroy.c +++ b/src/lxc/tools/lxc_destroy.c @@ -67,6 +67,7 @@ static bool do_destroy_with_snapshots(struct lxc_container *c); int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; bool bret; if (lxc_arguments_parse(&my_args, argc, argv)) @@ -75,8 +76,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); if (my_args.quiet) diff --git a/src/lxc/tools/lxc_device.c b/src/lxc/tools/lxc_device.c index 39c560cb9..6ce769a6a 100644 --- a/src/lxc/tools/lxc_device.c +++ b/src/lxc/tools/lxc_device.c @@ -101,6 +101,7 @@ static bool is_interface(const char* dev_name, pid_t pid) int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; char *cmd, *dev_name, *dst_name; bool ret = false; @@ -115,8 +116,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) goto err; lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_execute.c b/src/lxc/tools/lxc_execute.c index f26105ad6..5eeee0333 100644 --- a/src/lxc/tools/lxc_execute.c +++ b/src/lxc/tools/lxc_execute.c @@ -40,6 +40,7 @@ #include "config.h" #include "start.h" #include "utils.h" +#include "lxccontainer.h" lxc_log_define(lxc_execute_ui, lxc); @@ -106,6 +107,7 @@ int main(int argc, char *argv[]) { char *rcfile; struct lxc_conf *conf; + struct lxc_log log; int ret; lxc_list_init(&defines); @@ -116,8 +118,14 @@ int main(int argc, char *argv[]) if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_freeze.c b/src/lxc/tools/lxc_freeze.c index 33445c89e..3db1b6a4f 100644 --- a/src/lxc/tools/lxc_freeze.c +++ b/src/lxc/tools/lxc_freeze.c @@ -57,6 +57,7 @@ Options :\n\ int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); @@ -64,8 +65,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_info.c b/src/lxc/tools/lxc_info.c index c977f299c..bb919b5e5 100644 --- a/src/lxc/tools/lxc_info.c +++ b/src/lxc/tools/lxc_info.c @@ -394,6 +394,7 @@ static int print_info(const char *name, const char *lxcpath) int main(int argc, char *argv[]) { int ret = EXIT_FAILURE; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(ret); @@ -401,8 +402,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(ret); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_init.c b/src/lxc/tools/lxc_init.c index 1aaa96cd6..456d3fb03 100644 --- a/src/lxc/tools/lxc_init.c +++ b/src/lxc/tools/lxc_init.c @@ -37,6 +37,7 @@ #include "caps.h" #include "error.h" #include "initutils.h" +#include "lxccontainer.h" lxc_log_define(lxc_init, lxc); @@ -83,6 +84,7 @@ int main(int argc, char *argv[]) int i, have_status = 0, shutdown = 0; int opt; char *lxcpath = NULL, *name = NULL, *logpriority = NULL; + struct lxc_log log; while ((opt = getopt_long(argc, argv, "n:l:qP:", options, NULL)) != -1) { switch(opt) { @@ -94,7 +96,7 @@ int main(int argc, char *argv[]) break; case 'q': quiet = 1; - break; + break; case 'P': lxcpath = optarg; break; @@ -104,8 +106,14 @@ int main(int argc, char *argv[]) } } - err = lxc_log_init(name, name ? NULL : "none", logpriority, - basename(argv[0]), quiet, lxcpath); + log.name = name; + log.file = name ? NULL : "none"; + log.priority = logpriority; + log.prefix = basename(argv[0]); + log.quiet = quiet; + log.lxcpath = lxcpath; + + err = lxc_log_init(&log); if (err < 0) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_ls.c b/src/lxc/tools/lxc_ls.c index 63053b110..28d423909 100644 --- a/src/lxc/tools/lxc_ls.c +++ b/src/lxc/tools/lxc_ls.c @@ -203,6 +203,7 @@ Options :\n\ int main(int argc, char *argv[]) { int ret = EXIT_FAILURE; + struct lxc_log log; /* * The lxc parser requires that my_args.name is set. So let's satisfy * that condition by setting a dummy name which is never used. @@ -218,8 +219,14 @@ int main(int argc, char *argv[]) * We set the first argument that usually takes my_args.name to NULL so * that the log is only used when the user specifies a file. */ - if (lxc_log_init(NULL, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = NULL; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_monitor.c b/src/lxc/tools/lxc_monitor.c index d9da9b8bc..805041348 100644 --- a/src/lxc/tools/lxc_monitor.c +++ b/src/lxc/tools/lxc_monitor.c @@ -35,6 +35,7 @@ #include "log.h" #include "monitor.h" #include "arguments.h" +#include "lxccontainer.h" lxc_log_define(lxc_monitor_ui, lxc); @@ -91,6 +92,7 @@ int main(int argc, char *argv[]) struct pollfd *fds; nfds_t nfds; int len, rc_main, rc_snp, i; + struct lxc_log log; rc_main = EXIT_FAILURE; @@ -100,8 +102,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(rc_main); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_snapshot.c b/src/lxc/tools/lxc_snapshot.c index 6948d1988..ee0bff858 100644 --- a/src/lxc/tools/lxc_snapshot.c +++ b/src/lxc/tools/lxc_snapshot.c @@ -81,6 +81,7 @@ static void print_file(char *path); int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; int ret; if (lxc_arguments_parse(&my_args, argc, argv)) @@ -89,8 +90,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_start.c b/src/lxc/tools/lxc_start.c index 3c17ea0f9..469732db1 100644 --- a/src/lxc/tools/lxc_start.c +++ b/src/lxc/tools/lxc_start.c @@ -205,6 +205,7 @@ int main(int argc, char *argv[]) { int err = EXIT_FAILURE; struct lxc_conf *conf; + struct lxc_log log; char *const *args; char *rcfile = NULL; char *const default_args[] = { @@ -226,8 +227,14 @@ int main(int argc, char *argv[]) else args = my_args.argv; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(err); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_stop.c b/src/lxc/tools/lxc_stop.c index e2d64fb5c..4dfb65414 100644 --- a/src/lxc/tools/lxc_stop.c +++ b/src/lxc/tools/lxc_stop.c @@ -155,14 +155,21 @@ out: int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; bool s; int ret = EXIT_FAILURE; if (lxc_arguments_parse(&my_args, argc, argv)) exit(ret); - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(ret); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_unfreeze.c b/src/lxc/tools/lxc_unfreeze.c index 539654dfc..8461eb19e 100644 --- a/src/lxc/tools/lxc_unfreeze.c +++ b/src/lxc/tools/lxc_unfreeze.c @@ -55,6 +55,7 @@ Options :\n\ int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); @@ -62,8 +63,15 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override(); diff --git a/src/lxc/tools/lxc_wait.c b/src/lxc/tools/lxc_wait.c index de95696aa..7ab22b5d8 100644 --- a/src/lxc/tools/lxc_wait.c +++ b/src/lxc/tools/lxc_wait.c @@ -83,6 +83,7 @@ Options :\n\ int main(int argc, char *argv[]) { struct lxc_container *c; + struct lxc_log log; if (lxc_arguments_parse(&my_args, argc, argv)) exit(EXIT_FAILURE); @@ -90,8 +91,14 @@ int main(int argc, char *argv[]) if (!my_args.log_file) my_args.log_file = "none"; - if (lxc_log_init(my_args.name, my_args.log_file, my_args.log_priority, - my_args.progname, my_args.quiet, my_args.lxcpath[0])) + log.name = my_args.name; + log.file = my_args.log_file; + log.priority = my_args.log_priority; + log.prefix = my_args.progname; + log.quiet = my_args.quiet; + log.lxcpath = my_args.lxcpath[0]; + + if (lxc_log_init(&log)) exit(EXIT_FAILURE); lxc_log_options_no_override();