From: Lennart Poettering Date: Thu, 11 Jan 2018 12:15:19 +0000 (+0100) Subject: log: assign the realm to a variable first in log_full_errno_realm() X-Git-Tag: v237~107^2~2 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=c1e83af84eea52c22f40ae8f34b1c5a32d94a90d;p=thirdparty%2Fsystemd.git log: assign the realm to a variable first in log_full_errno_realm() Let's avoid duplicate evaluation (which might have side-effects) and assign the realm to an internal variable first, as we need it twice. --- diff --git a/src/basic/log.h b/src/basic/log.h index 811e5a6cc54..b97cdcd703d 100644 --- a/src/basic/log.h +++ b/src/basic/log.h @@ -238,9 +238,9 @@ void log_assert_failed_return_realm( /* Logging with level */ #define log_full_errno_realm(realm, level, error, ...) \ ({ \ - int _level = (level), _e = (error); \ - (log_get_max_level_realm((realm)) >= LOG_PRI(_level)) \ - ? log_internal_realm(LOG_REALM_PLUS_LEVEL((realm), _level), _e, \ + int _level = (level), _e = (error), _realm = (realm); \ + (log_get_max_level_realm(_realm) >= LOG_PRI(_level)) \ + ? log_internal_realm(LOG_REALM_PLUS_LEVEL(_realm, _level), _e, \ __FILE__, __LINE__, __func__, __VA_ARGS__) \ : -abs(_e); \ })