diff options
| author | Andrey Konovalov <andreyknvl@google.com> | 2020-05-28 21:31:58 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2020-05-29 09:50:00 +0200 |
| commit | 6c68daf0988d1474ff10ace571a9a7e0d1f27f89 (patch) | |
| tree | 39d3fe2caf692c1aee10a48c59d6083a49358b54 /executor | |
| parent | 7e7ceb21ba30cacae05f29b2f574b9175ccfd425 (diff) | |
executor: fix printing null master in netlink_device_change
The issues is only present with verbose debugging enabled.
executor/common_linux.h: In function ‘void netlink_device_change(nlmsg*, int, const char*, bool, const char*, const void*, int, const char*)’:
executor/common_linux.h:380:7: error: ‘%s’ directive argument is null [-Werror=format-overflow=]
380 | debug("netlink: device %s up master %s: %s\n", name, master, strerror(err));
Diffstat (limited to 'executor')
| -rw-r--r-- | executor/common_linux.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/executor/common_linux.h b/executor/common_linux.h index ec4dba32c..b70729987 100644 --- a/executor/common_linux.h +++ b/executor/common_linux.h @@ -377,7 +377,7 @@ static void netlink_device_change(struct nlmsg* nlmsg, int sock, const char* nam if (macsize) netlink_attr(nlmsg, IFLA_ADDRESS, mac, macsize); int err = netlink_send(nlmsg, sock); - debug("netlink: device %s up master %s: %s\n", name, master, strerror(err)); + debug("netlink: device %s up master %s: %s\n", name, master ? master : "NULL", strerror(err)); (void)err; } #endif |