From a05b04c0fdba89cc94ab7b187b674cea86c46db2 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Sun, 11 Jan 2015 20:00:54 +0100 Subject: [PATCH 1/5] dhcp/trivial: remove shaddowed '#if 0 /* NM_IGNORED */' block Remove '#if 0' inside another '#if 0' block. --- src/dhcp-manager/systemd-dhcp/src/shared/util.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/util.c b/src/dhcp-manager/systemd-dhcp/src/shared/util.c index 05d4eb757a..f3009df333 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/util.c @@ -3342,7 +3342,6 @@ fail: return NULL; } -#if 0 /* NM_IGNORED */ char **replace_env_argv(char **argv, char **env) { char **ret, **i; unsigned k = 0, l = 0; @@ -3407,7 +3406,6 @@ char **replace_env_argv(char **argv, char **env) { ret[k] = NULL; return ret; } -#endif int fd_columns(int fd) { struct winsize ws = {}; From 94b9c79cf67a09ab79a3826ed46a4cc8150ca5cc Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Sun, 11 Jan 2015 19:54:45 +0100 Subject: [PATCH 2/5] dhcp/trivial: add '/* NM_IGNORED */' comment to #endif added by NetworkManager --- .../src/libsystemd-network/network-internal.c | 8 ++-- .../src/libsystemd-network/network-internal.h | 2 +- .../src/libsystemd-network/sd-dhcp-lease.c | 2 +- .../src/libsystemd-network/sd-dhcp6-client.c | 6 +-- .../systemd-dhcp/src/shared/macro.h | 2 +- .../systemd-dhcp/src/shared/strv.c | 2 +- .../systemd-dhcp/src/shared/time-util.c | 6 +-- .../systemd-dhcp/src/shared/util.c | 44 +++++++++---------- .../systemd-dhcp/src/shared/util.h | 8 ++-- 9 files changed, 40 insertions(+), 40 deletions(-) diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c index b9334bd657..f1e5589b48 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c @@ -32,17 +32,17 @@ #include "strv.h" #include "siphash24.h" #include "libudev-private.h" -#endif +#endif /* NM_IGNORED */ #include "dhcp-lease-internal.h" #if 0 /* NM_IGNORED */ #include "log.h" #include "utf8.h" -#endif +#endif /* NM_IGNORED */ #include "util.h" #if 0 /* NM_IGNORED */ #include "conf-parser.h" #include "condition.h" -#endif +#endif /* NM_IGNORED */ #include "network-internal.h" #if 0 /* NM_IGNORED */ @@ -306,7 +306,7 @@ int config_parse_hwaddr(const char *unit, return 0; } -#endif +#endif /* NM_IGNORED */ void serialize_in_addrs(FILE *f, const struct in_addr *addresses, size_t size) { unsigned i; diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.h b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.h index f87c863d91..7a482f2043 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.h +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.h @@ -65,7 +65,7 @@ int config_parse_ifalias(const char *unit, const char *filename, unsigned line, int net_get_unique_predictable_data(struct udev_device *device, uint8_t result[8]); const char *net_get_name(struct udev_device *device); -#endif +#endif /* NM_IGNORED */ void serialize_in_addrs(FILE *f, const struct in_addr *addresses, size_t size); int deserialize_in_addrs(struct in_addr **addresses, const char *string); diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c index 2d13d503e1..363c3da37c 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c @@ -34,7 +34,7 @@ #include "list.h" #if 0 /* NM_IGNORED */ #include "mkdir.h" -#endif +#endif /* NM_IGNORED */ #include "fileio.h" #include "unaligned.h" #include "in-addr-util.h" diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c index 6c5a6ab536..ef24774650 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c @@ -32,7 +32,7 @@ #include "udev.h" #include "udev-util.h" #include "virt.h" -#endif +#endif /* NM_IGNORED */ #include "siphash24.h" #include "util.h" #include "refcnt.h" @@ -1180,7 +1180,7 @@ int sd_dhcp6_client_new(sd_dhcp6_client **ret) #if 0 /* NM_IGNORED */ sd_id128_t machine_id; int r; -#endif +#endif /* NM_IGNORED */ size_t t; assert_return(ret, -EINVAL); @@ -1210,7 +1210,7 @@ int sd_dhcp6_client_new(sd_dhcp6_client **ret) /* a bit of snake-oil perhaps, but no need to expose the machine-id directly */ siphash24(client->duid.en.id, &machine_id, sizeof(machine_id), HASH_KEY.bytes); -#endif +#endif /* NM_IGNORED */ client->req_opts_len = ELEMENTSOF(default_req_opts); diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/macro.h b/src/dhcp-manager/systemd-dhcp/src/shared/macro.h index e6cf6eecd7..24d4e6cdd0 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/macro.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/macro.h @@ -410,4 +410,4 @@ do { \ #endif #include "log.h" -#endif +#endif /* NM_IGNORED */ diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/strv.c b/src/dhcp-manager/systemd-dhcp/src/shared/strv.c index be2bd51106..dd0073d68a 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/strv.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/strv.c @@ -312,7 +312,7 @@ int strv_split_quoted(char ***t, const char *s, bool relax) { return 0; } -#endif +#endif /* NM_IGNORED */ char *strv_join(char **l, const char *separator) { char *r, *e; diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c index c8fa1acba6..0726c376ec 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c @@ -32,7 +32,7 @@ #include "time-util.h" #if 0 /* NM_IGNORED */ #include "strv.h" -#endif +#endif /* NM_IGNORED */ usec_t now(clockid_t clock_id) { struct timespec ts; @@ -279,7 +279,7 @@ char *format_timestamp_relative(char *buf, size_t l, usec_t t) { buf[l-1] = 0; return buf; } -#endif +#endif /* NM_IGNORED */ char *format_timespan(char *buf, size_t l, usec_t t, usec_t accuracy) { static const struct { @@ -981,7 +981,7 @@ bool timezone_is_valid(const char *name) { return true; } -#endif +#endif /* NM_IGNORED */ clockid_t clock_boottime_or_monotonic(void) { static clockid_t clock = -1; diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/util.c b/src/dhcp-manager/systemd-dhcp/src/shared/util.c index f3009df333..381a8d7b25 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/util.c @@ -90,7 +90,7 @@ #include "gunicode.h" #include "virt.h" #include "def.h" -#endif +#endif /* NM_IGNORED */ #if 0 /* NM_IGNORED */ int saved_argc = 0; @@ -112,7 +112,7 @@ size_t page_size(void) { pgsz = (size_t) r; return pgsz; } -#endif +#endif /* NM_IGNORED */ bool streq_ptr(const char *a, const char *b) { @@ -311,7 +311,7 @@ int parse_uid(const char *s, uid_t* ret_uid) { *ret_uid = uid; return 0; } -#endif +#endif /* NM_IGNORED */ int safe_atou(const char *s, unsigned *ret_u) { char *x = NULL; @@ -466,7 +466,7 @@ int safe_atod(const char *s, double *ret_d) { *ret_d = (double) d; return 0; } -#endif +#endif /* NM_IGNORED */ static size_t strcspn_escaped(const char *s, const char *reject) { bool escaped = false; @@ -617,7 +617,7 @@ int get_starttime_of_pid(pid_t pid, unsigned long long *st) { return 0; } -#endif +#endif /* NM_IGNORED */ int fchmod_umask(int fd, mode_t m) { mode_t u; @@ -875,7 +875,7 @@ int get_process_gid(pid_t pid, gid_t *gid) { assert_cc(sizeof(uid_t) == sizeof(gid_t)); return get_process_id(pid, "Gid:", gid); } -#endif +#endif /* NM_IGNORED */ char *strnappend(const char *s, const char *suffix, size_t b) { size_t a; @@ -1049,7 +1049,7 @@ int reset_signal_mask(void) { return 0; } -#endif +#endif /* NM_IGNORED */ char *strstrip(char *s) { char *e; @@ -1158,7 +1158,7 @@ int rmdir_parents(const char *path, const char *stop) { return 0; } -#endif +#endif /* NM_IGNORED */ char hexchar(int x) { static const char table[16] = "0123456789abcdef"; @@ -1646,7 +1646,7 @@ int close_all_fds(const int except[], unsigned n_except) { return r; } -#endif +#endif /* NM_IGNORED */ bool chars_intersect(const char *a, const char *b) { const char *p; @@ -2269,7 +2269,7 @@ void safe_close_pair(int p[]) { p[0] = safe_close(p[0]); p[1] = safe_close(p[1]); } -#endif +#endif /* NM_IGNORED */ ssize_t loop_read(int fd, void *buf, size_t nbytes, bool do_poll) { uint8_t *p = buf; @@ -2551,7 +2551,7 @@ char* dirname_malloc(const char *path) { return dir; } -#endif +#endif /* NM_IGNORED */ int dev_urandom(void *p, size_t n) { #if 0 /* NM_IGNORED */ @@ -2597,7 +2597,7 @@ int dev_urandom(void *p, size_t n) { #else /* NM IGNORED */ int fd; ssize_t k; -#endif +#endif /* NM_IGNORED */ fd = open("/dev/urandom", O_RDONLY|O_CLOEXEC|O_NOCTTY); if (fd < 0) @@ -4169,7 +4169,7 @@ char* strshorten(char *s, size_t l) { return s; } -#endif +#endif /* NM_IGNORED */ static bool hostname_valid_char(char c) { return @@ -4272,7 +4272,7 @@ int pipe_eof(int fd) { return pollfd.revents & POLLHUP; } -#endif +#endif /* NM_IGNORED */ int fd_wait_for_event(int fd, int event, usec_t t) { @@ -4805,7 +4805,7 @@ int get_files_in_directory(const char *path, char ***list) { return n; } -#endif +#endif /* NM_IGNORED */ char *strjoin(const char *x, ...) { va_list ap; @@ -5172,7 +5172,7 @@ finish: return buf; } -#endif +#endif /* NM_IGNORED */ void* memdup(const void *p, size_t l) { void *r; @@ -5619,7 +5619,7 @@ bool string_is_safe(const char *p) { return true; } -#endif +#endif /* NM_IGNORED */ /** * Check if a string contains control characters. If 'ok' is non-NULL @@ -6122,7 +6122,7 @@ char *strrep(const char *s, unsigned n) { *p = 0; return r; } -#endif +#endif /* NM_IGNORED */ void* greedy_realloc(void **p, size_t *allocated, size_t need, size_t size) { size_t a, newalloc; @@ -6538,7 +6538,7 @@ int getpeersec(int fd, char **ret) { *ret = s; return 0; } -#endif +#endif /* NM_IGNORED */ /* This is much like like mkostemp() but is subject to umask(). */ int mkostemp_safe(char *pattern, int flags) { @@ -6943,7 +6943,7 @@ int bind_remount_recursive(const char *prefix, bool ro) { } } } -#endif +#endif /* NM_IGNORED */ int fflush_and_check(FILE *f) { assert(f); @@ -7005,7 +7005,7 @@ char *tempfn_random(const char *p) { return t; } -#endif +#endif /* NM_IGNORED */ /* make sure the hostname is not "localhost" */ bool is_localhost(const char *hostname) { @@ -7344,5 +7344,5 @@ int sethostname_idempotent(const char *s) { return 1; } -#endif +#endif /* NM_IGNORED */ diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/util.h b/src/dhcp-manager/systemd-dhcp/src/shared/util.h index a7690655eb..68b08c4016 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/util.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/util.h @@ -85,12 +85,12 @@ #else # error Unknown rlim_t size #endif -#endif +#endif /* NM_IGNORED */ #include "macro.h" #if 0 /* NM_IGNORED */ #include "missing.h" -#endif +#endif /* NM_IGNORED */ #include "time-util.h" /* What is interpreted as whitespace? */ @@ -987,7 +987,7 @@ bool pid_is_unwaited(pid_t pid); #if 0 /* NM_IGNORED */ int getpeercred(int fd, struct ucred *ucred); int getpeersec(int fd, char **ret); -#endif +#endif /* NM_IGNORED */ int writev_safe(int fd, const struct iovec *w, int j); @@ -1010,7 +1010,7 @@ union file_handle_union { struct file_handle handle; char padding[sizeof(struct file_handle) + MAX_HANDLE_SZ]; }; -#endif +#endif /* NM_IGNORED */ int update_reboot_param_file(const char *param); From 5162426d416a4d309602d695c5966d109fc7bcc0 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Fri, 5 Dec 2014 00:54:30 +0100 Subject: [PATCH 3/5] logging: add error argument to _nm_log() to support "%m" format specifier A gnu extension to printf adds the format specifier "%m" to print @errno. To preserve the error number until the point where the logging statement is constructed, pass it as an additional argument to _nm_log(). This is not (yet) used from NM internal code. But systemd is adding similar functionality to its logging functions. Add the same also to nm-logging, to support systemd's usage of "%m". --- src/dhcp-manager/systemd-dhcp/nm-sd-adapt.h | 2 +- src/nm-logging.c | 5 +++++ src/nm-logging.h | 5 +++-- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/src/dhcp-manager/systemd-dhcp/nm-sd-adapt.h b/src/dhcp-manager/systemd-dhcp/nm-sd-adapt.h index ce1716b175..a95f5bfb42 100644 --- a/src/dhcp-manager/systemd-dhcp/nm-sd-adapt.h +++ b/src/dhcp-manager/systemd-dhcp/nm-sd-adapt.h @@ -57,7 +57,7 @@ G_STMT_START { \ if (nm_logging_enabled (_l, LOGD_DHCP)) { \ const char *_location = strrchr (file "", '/'); \ \ - _nm_log (_location ? _location + 1 : file, line, func, _l, LOGD_DHCP, format, ## __VA_ARGS__); \ + _nm_log (_location ? _location + 1 : file, line, func, _l, LOGD_DHCP, 0, format, ## __VA_ARGS__); \ } \ } G_STMT_END diff --git a/src/nm-logging.c b/src/nm-logging.c index 7f2170ca39..1e9d422f2a 100644 --- a/src/nm-logging.c +++ b/src/nm-logging.c @@ -365,6 +365,7 @@ _nm_log (const char *file, const char *func, NMLogLevel level, NMLogDomain domain, + int error, const char *fmt, ...) { @@ -382,6 +383,10 @@ _nm_log (const char *file, if (!(logging[level] & domain)) return; + /* Make sure that %m maps to the specified error */ + if (error != 0) + errno = error; + va_start (args, fmt); msg = g_strdup_vprintf (fmt, args); va_end (args); diff --git a/src/nm-logging.h b/src/nm-logging.h index 4ab2db3ad5..7417dfa0f7 100644 --- a/src/nm-logging.h +++ b/src/nm-logging.h @@ -105,7 +105,7 @@ typedef enum { /*< skip >*/ #define nm_log(level, domain, ...) \ G_STMT_START { \ if (nm_logging_enabled ((level), (domain))) { \ - _nm_log (__FILE__, __LINE__, G_STRFUNC, (level), (domain), __VA_ARGS__); \ + _nm_log (__FILE__, __LINE__, G_STRFUNC, (level), (domain), 0, __VA_ARGS__); \ } \ } G_STMT_END @@ -139,8 +139,9 @@ void _nm_log (const char *file, const char *func, NMLogLevel level, NMLogDomain domain, + int error, const char *fmt, - ...) __attribute__((__format__ (__printf__, 6, 7))); + ...) __attribute__((__format__ (__printf__, 7, 8))); const char *nm_logging_level_to_string (void); const char *nm_logging_domains_to_string (void); From 15bebd7c5195957315b68e0554685c3232c5ea1e Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Mon, 2 Feb 2015 12:13:56 +0100 Subject: [PATCH 4/5] dhcp: don't include config.h in systemd source systemd sources should all include 'nm-sd-adapt.h' as first header, which in turn already includes our 'config.h'. --- .../systemd-dhcp/src/libsystemd-network/dhcp-network.c | 2 -- .../systemd-dhcp/src/libsystemd-network/dhcp-option.c | 2 -- .../systemd-dhcp/src/libsystemd-network/dhcp-packet.c | 2 -- .../systemd-dhcp/src/libsystemd-network/dhcp6-network.c | 2 -- .../systemd-dhcp/src/libsystemd-network/dhcp6-option.c | 2 -- .../systemd-dhcp/src/libsystemd-network/network-internal.c | 2 -- .../systemd-dhcp/src/libsystemd-network/sd-dhcp-client.c | 2 -- .../systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c | 2 -- .../systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c | 2 -- .../systemd-dhcp/src/libsystemd-network/sd-dhcp6-lease.c | 2 -- src/dhcp-manager/systemd-dhcp/src/shared/fileio.c | 2 +- src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.c | 2 +- src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c | 2 +- src/dhcp-manager/systemd-dhcp/src/shared/strv.c | 2 -- src/dhcp-manager/systemd-dhcp/src/shared/time-util.c | 2 -- src/dhcp-manager/systemd-dhcp/src/shared/utf8.c | 2 -- src/dhcp-manager/systemd-dhcp/src/shared/util.c | 2 -- 17 files changed, 3 insertions(+), 31 deletions(-) diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-network.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-network.c index 971b3aa180..26e6e9f3df 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-network.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-network.c @@ -17,8 +17,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-option.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-option.c index 9759e575a2..50ecf87383 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-option.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-option.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-packet.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-packet.c index 5bb065ba1c..e6ebf86ae1 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-packet.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-packet.c @@ -18,8 +18,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-network.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-network.c index fbb20ef94b..626a4d6a7f 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-network.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-network.c @@ -17,8 +17,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-option.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-option.c index 3d6e24623a..10872a3159 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-option.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-option.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c index f1e5589b48..8251269a0a 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/network-internal.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-client.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-client.c index 3e4cd8d9e7..d5ada92375 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-client.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-client.c @@ -17,8 +17,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c index 363c3da37c..6a28f9b01f 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp-lease.c @@ -18,8 +18,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c index ef24774650..792e46d57a 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-client.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-lease.c b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-lease.c index 0df6984d60..6a1cb22daa 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-lease.c +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/sd-dhcp6-lease.c @@ -18,8 +18,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/fileio.c b/src/dhcp-manager/systemd-dhcp/src/shared/fileio.c index 0757d5d977..84fd50495f 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/fileio.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/fileio.c @@ -19,7 +19,7 @@ along with systemd; If not, see . ***/ -#include "config.h" +#include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.c b/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.c index 27edeb8d93..530be45e14 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.c @@ -19,7 +19,7 @@ along with systemd; If not, see . ***/ -#include "config.h" +#include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c index f9e59c6f59..f6757e4c3e 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c @@ -14,7 +14,7 @@ (Minimal changes made by Lennart Poettering, to make clean for inclusion in systemd) */ -#include "config.h" +#include "nm-sd-adapt.h" #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/strv.c b/src/dhcp-manager/systemd-dhcp/src/shared/strv.c index dd0073d68a..74a080ff16 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/strv.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/strv.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c index 0726c376ec..5d79f15dc0 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/utf8.c b/src/dhcp-manager/systemd-dhcp/src/shared/utf8.c index cfedbfcb99..261f39db84 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/utf8.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/utf8.c @@ -43,8 +43,6 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "config.h" - #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/util.c b/src/dhcp-manager/systemd-dhcp/src/shared/util.c index 381a8d7b25..010243b219 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/util.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/util.c @@ -19,8 +19,6 @@ along with systemd; If not, see . ***/ -#include "config.h" - #include "nm-sd-adapt.h" #include From 42b39b948297052b574a0735ec3fa29726a8fcee Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Mon, 2 Feb 2015 12:54:13 +0100 Subject: [PATCH 5/5] dhcp: ensure every systemd dhcp file includes 'nm-sd-adapt.h' first --- .../systemd-dhcp/src/libsystemd-network/dhcp-lease-internal.h | 2 ++ .../systemd-dhcp/src/libsystemd-network/dhcp-protocol.h | 2 ++ .../systemd-dhcp/src/libsystemd-network/dhcp6-protocol.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/async.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/fileio.h | 3 +++ src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/list.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/macro.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/refcnt.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c | 1 + src/dhcp-manager/systemd-dhcp/src/shared/siphash24.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/socket-util.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/sparse-endian.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/strv.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/time-util.h | 4 ++-- src/dhcp-manager/systemd-dhcp/src/shared/unaligned.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/shared/utf8.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/_sd-common.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-client.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-lease.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-client.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-lease.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-event.h | 2 ++ src/dhcp-manager/systemd-dhcp/src/systemd/sd-id128.h | 2 ++ 24 files changed, 48 insertions(+), 2 deletions(-) diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-lease-internal.h b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-lease-internal.h index 9e184ac4b5..2d7bf03bab 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-lease-internal.h +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-lease-internal.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-protocol.h b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-protocol.h index abca9422c5..da483feadf 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-protocol.h +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp-protocol.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-protocol.h b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-protocol.h index eaa671711f..e46470d20e 100644 --- a/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-protocol.h +++ b/src/dhcp-manager/systemd-dhcp/src/libsystemd-network/dhcp6-protocol.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/async.h b/src/dhcp-manager/systemd-dhcp/src/shared/async.h index 7f1ef79532..44b886a661 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/async.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/async.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + int asynchronous_job(void* (*func)(void *p), void *arg); int asynchronous_sync(void); diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/fileio.h b/src/dhcp-manager/systemd-dhcp/src/shared/fileio.h index 5ae51c1e28..fccea7725a 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/fileio.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/fileio.h @@ -20,6 +20,9 @@ You should have received a copy of the GNU Lesser General Public License along with systemd; If not, see . ***/ + +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.h b/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.h index 8da030ceb6..36afee2c42 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/in-addr-util.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include "macro.h" diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/list.h b/src/dhcp-manager/systemd-dhcp/src/shared/list.h index c020f7e936..dcb3e0d94c 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/list.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/list.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + /* The head of the linked list. Use this in the structure that shall * contain the head of the linked list */ #define LIST_HEAD(t,name) \ diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/macro.h b/src/dhcp-manager/systemd-dhcp/src/shared/macro.h index 24d4e6cdd0..82469d81f0 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/macro.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/macro.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/refcnt.h b/src/dhcp-manager/systemd-dhcp/src/shared/refcnt.h index 0502c20a2e..e6e9a6bd6e 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/refcnt.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/refcnt.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + /* A type-safe atomic refcounter */ typedef struct { diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c index f6757e4c3e..a26f8e28ca 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c +++ b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.c @@ -14,6 +14,7 @@ (Minimal changes made by Lennart Poettering, to make clean for inclusion in systemd) */ + #include "nm-sd-adapt.h" #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.h b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.h index 62e1168a79..3450d754fe 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/siphash24.h @@ -1,5 +1,7 @@ #pragma once +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/socket-util.h b/src/dhcp-manager/systemd-dhcp/src/shared/socket-util.h index 07d0aff72b..550ed9853b 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/socket-util.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/socket-util.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/sparse-endian.h b/src/dhcp-manager/systemd-dhcp/src/shared/sparse-endian.h index c913fda8c5..02de0feca8 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/sparse-endian.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/sparse-endian.h @@ -21,6 +21,8 @@ #ifndef SPARSE_ENDIAN_H #define SPARSE_ENDIAN_H +#include "nm-sd-adapt.h" + #include #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/strv.h b/src/dhcp-manager/systemd-dhcp/src/shared/strv.h index 47618bd26c..15ef0f1740 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/strv.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/strv.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.h b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.h index 578a0c1ca4..e3e84acbae 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/time-util.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/time-util.h @@ -21,14 +21,14 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include typedef uint64_t usec_t; typedef uint64_t nsec_t; -#include "nm-sd-adapt.h" - #define NSEC_FMT "%" PRIu64 #define USEC_FMT "%" PRIu64 diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/unaligned.h b/src/dhcp-manager/systemd-dhcp/src/shared/unaligned.h index d6181dd9a9..3b73a9c682 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/unaligned.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/unaligned.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include static inline uint16_t unaligned_read_be16(const void *_u) { diff --git a/src/dhcp-manager/systemd-dhcp/src/shared/utf8.h b/src/dhcp-manager/systemd-dhcp/src/shared/utf8.h index c087995930..d7728a162f 100644 --- a/src/dhcp-manager/systemd-dhcp/src/shared/utf8.h +++ b/src/dhcp-manager/systemd-dhcp/src/shared/utf8.h @@ -21,6 +21,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include "macro.h" diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/_sd-common.h b/src/dhcp-manager/systemd-dhcp/src/systemd/_sd-common.h index 896a027eb5..abcd57dbe4 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/_sd-common.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/_sd-common.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + /* This is a private header; never even think of including this directly! */ #if __INCLUDE_LEVEL__ <= 1 diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-client.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-client.h index 951662e56c..527c4239d1 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-client.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-client.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-lease.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-lease.h index 4296b91d8a..80d32134b1 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-lease.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp-lease.h @@ -23,6 +23,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-client.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-client.h index 38c1c3fb12..301f306705 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-client.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-client.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include "sd-event.h" diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-lease.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-lease.h index 1126f1ac2e..e72eb8be05 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-lease.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-dhcp6-lease.h @@ -23,6 +23,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include typedef struct sd_dhcp6_lease sd_dhcp6_lease; diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-event.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-event.h index 25a10f99ab..00bc0fc77e 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-event.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-event.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include #include diff --git a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-id128.h b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-id128.h index 4fdbf3ab41..df9b83c4c0 100644 --- a/src/dhcp-manager/systemd-dhcp/src/systemd/sd-id128.h +++ b/src/dhcp-manager/systemd-dhcp/src/systemd/sd-id128.h @@ -22,6 +22,8 @@ along with systemd; If not, see . ***/ +#include "nm-sd-adapt.h" + #include #include