From 65c629f28ff4ab42889acbf60ddaaf5aacf5f399 Mon Sep 17 00:00:00 2001
From: =?utf8?q?=C5=81ukasz=20Stelmach?= <l.stelmach@samsung.com>
Date: Fri, 10 Sep 2021 20:02:53 +0200
Subject: [PATCH] logger: handle error values from filp_open()
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

filp_open() may return -errno, so check for that instead of checking
for NULL.

Change-Id: I5a694555bcd082506330e08f48ed4b7043c4e120
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
---
 kernel/logger.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/kernel/logger.c b/kernel/logger.c
index 202703c..eb287e4 100644
--- a/kernel/logger.c
+++ b/kernel/logger.c
@@ -484,9 +484,8 @@ static struct file *make_new_file(struct file *file)
 
 	nfile = filp_open(p, O_WRONLY, 0);
 	kfree(pbuf);
-	if (!nfile) {
-		return ERR_PTR(-EFAULT);
-	}
+	if (IS_ERR(nfile))
+		return nfile;
 
 	nwriter = nfile->private_data;
 	nwriter->prio = writer->prio;
-- 
2.7.4