From d05a3d937ddc2b666b153e6adaf9610800772b68 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Fri, 31 Jan 2020 17:41:08 -0500 Subject: [PATCH] Fix: unchecked return value of lttng_directory_handle_create() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Although unlikely in this case (as the path was just created), the creation of a directory handle can fail and this should always be accounted-for. 1415129 Dereference null return value If the function actually returns a null value, a null pointer dereference will occur. In lttng_unlinked_file_pool_add_inode: Return value of function which returns null is dereferenced without checking (CWE-476) Reported-by: Coverity Scan Signed-off-by: Jérémie Galarneau Change-Id: I914802c25b569320e08265aae251e062bccbd50e --- src/common/fd-tracker/inode.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/common/fd-tracker/inode.c b/src/common/fd-tracker/inode.c index 727f6141a..b894f8fbf 100644 --- a/src/common/fd-tracker/inode.c +++ b/src/common/fd-tracker/inode.c @@ -133,6 +133,12 @@ static int lttng_unlinked_file_pool_add_inode( } pool->unlink_directory_handle = lttng_directory_handle_create( pool->unlink_directory_path); + if (!pool->unlink_directory_handle) { + ERR("Failed to create directory handle to unlinked file pool at %s", + pool->unlink_directory_path); + ret = -1; + goto end; + } } ret = lttng_directory_handle_rename(inode->location.directory_handle, -- 2.34.1