From: Łukasz Stelmach <l.stelmach@samsung.com>
Date: Wed, 15 Sep 2021 12:54:18 +0000 (+0200)
Subject: logger: flush data from stdio writers under mutex
X-Git-Tag: submit/tizen/20211126.100737~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dab1fe97e1899eb3aba08d02786ec75b7fa68384;p=platform%2Fkernel%2Flinux-tizen-modules-source.git

logger: flush data from stdio writers under mutex

Operations during flushing logs delivered via stdio need to be performed
with the mutex held.

Change-Id: I4655b0af75172f8f22b797c0caadd28cb4619cf0
Suggested-by: Mateusz Majewski <m.majewski2@samsung.com>
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
---

diff --git a/kernel/logger.c b/kernel/logger.c
index 48dbf4a..202703c 100644
--- a/kernel/logger.c
+++ b/kernel/logger.c
@@ -806,10 +806,14 @@ static int logger_release(struct inode *ignored, struct file *file)
 		kfree(reader);
 	} else {
 		struct logger_writer *writer = file->private_data;
-
+		struct logger_log *log = writer->log;
 		bool from_stdio = writer->tag && writer->prio >= 2;
-		if (from_stdio && writer->b_off > 0)
+
+		if (from_stdio && writer->b_off > 0){
+			mutex_lock(&log->mutex);
 			flush_thread_data(file);
+			mutex_unlock(&log->mutex);
+		}
 
 		kfree(writer->tag);
 		kfree(writer->buffer);