main - lvmdbusd: Correct locking for _common_log

[Tony Asleson]

Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=f7ca470abe995210e80080...
Commit:        f7ca470abe995210e8008084c0b4436292b2816b
Parent:        d6f8f59239c65a7f1f9be1d24bf9ba02cfee5b0e
Author:        Tony Asleson 
AuthorDate:    Thu Mar 30 10:07:13 2023 -0500
Committer:     Tony Asleson 
CommitterDate: Thu Mar 30 11:04:42 2023 -0500

lvmdbusd: Correct locking for _common_log

Reduce the lock time and include the flush in the lock.

Reported by: Zbigniew J��drzejewski-Szmek 
---
 daemons/lvmdbusd/utils.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/daemons/lvmdbusd/utils.py b/daemons/lvmdbusd/utils.py
index 6013b656f..fdd7578c1 100644
--- a/daemons/lvmdbusd/utils.py
+++ b/daemons/lvmdbusd/utils.py
@@ -323,16 +323,17 @@ def _format_log_entry(msg):

def _common_log(msg, *attributes):
-	cfg.stdout_lock.acquire()
    msg = _format_log_entry(msg)

+	cfg.stdout_lock.acquire()
+
    if STDOUT_TTY and attributes:
    	print(color(msg, *attributes))
    else:
    	print(msg)

-	cfg.stdout_lock.release()
    sys.stdout.flush()
+	cfg.stdout_lock.release()

# Serializes access to stdout to prevent interleaved output