diff options
author | Thomas Haller <thaller@redhat.com> | 2022-01-28 16:22:32 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2022-02-02 11:08:30 +0100 |
commit | 88e7ea20c3d1bfca9eb0629519655ea387384014 (patch) | |
tree | 20337f46d695547724a588b6fa9ca4a4fde9a2e5 | |
parent | 49fe45b155f3ce458ee329a595cb417f4fc3ff3c (diff) |
l3cfg: only constructing log message when needed in _l3_acd_data_state_set_full()
-rw-r--r-- | src/core/nm-l3cfg.c | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/src/core/nm-l3cfg.c b/src/core/nm-l3cfg.c index b42ff8cb75..0b94f8cb08 100644 --- a/src/core/nm-l3cfg.c +++ b/src/core/nm-l3cfg.c @@ -2071,17 +2071,19 @@ _nm_printf(5, 6) static void _l3_acd_data_state_set_full(NML3Cfg *self, else changed = FALSE; - if (format) { - gs_free char *msg = NULL; - va_list args; + if (_LOGT_ENABLED()) { + if (format) { + gs_free char *msg = NULL; + va_list args; - va_start(args, format); - msg = g_strdup_vprintf(format, args); - va_end(args); + va_start(args, format); + msg = g_strdup_vprintf(format, args); + va_end(args); - _LOGT_acd(acd_data, "set state to %s (%s)", _l3_acd_addr_state_to_string(state), msg); - } else - _LOGT_acd(acd_data, "set state to %s", _l3_acd_addr_state_to_string(state)); + _LOGT_acd(acd_data, "set state to %s (%s)", _l3_acd_addr_state_to_string(state), msg); + } else + _LOGT_acd(acd_data, "set state to %s", _l3_acd_addr_state_to_string(state)); + } if (changed && allow_commit) { /* The availability of an address just changed (and we are instructed to |