|
 |
28bab8 |
From 34b7778028ab60dd3079a2ad3e1df23e87c8708f Mon Sep 17 00:00:00 2001
|
|
|
28bab8 |
From: Ernestas Kulik <ekulik@redhat.com>
|
|
|
28bab8 |
Date: Wed, 29 May 2019 09:17:02 +0200
|
|
|
28bab8 |
Subject: [PATCH] =?UTF-8?q?lib:=20file=5Fobj:=20Don=E2=80=99t=20dereferenc?=
|
|
|
28bab8 |
=?UTF-8?q?e=20null=20pointers?=
|
|
|
28bab8 |
MIME-Version: 1.0
|
|
|
28bab8 |
Content-Type: text/plain; charset=UTF-8
|
|
|
28bab8 |
Content-Transfer-Encoding: 8bit
|
|
|
28bab8 |
|
|
|
28bab8 |
And, amazingly, someone forgot a semicolon after a return statement in
|
|
|
28bab8 |
such a fix.
|
|
|
28bab8 |
|
|
|
28bab8 |
Signed-off-by: Ernestas Kulik <ekulik@redhat.com>
|
|
|
28bab8 |
---
|
|
|
28bab8 |
src/lib/file_obj.c | 8 ++++++--
|
|
|
28bab8 |
1 file changed, 6 insertions(+), 2 deletions(-)
|
|
|
28bab8 |
|
|
|
28bab8 |
diff --git a/src/lib/file_obj.c b/src/lib/file_obj.c
|
|
|
28bab8 |
index bbb5b9f..aa973b1 100644
|
|
|
28bab8 |
--- a/src/lib/file_obj.c
|
|
|
28bab8 |
+++ b/src/lib/file_obj.c
|
|
|
28bab8 |
@@ -30,7 +30,7 @@ file_obj_t *new_file_obj(const char* fullpath, const char* filename)
|
|
|
28bab8 |
void free_file_obj(file_obj_t *f)
|
|
|
28bab8 |
{
|
|
|
28bab8 |
if (f == NULL)
|
|
|
28bab8 |
- return
|
|
|
28bab8 |
+ return;
|
|
|
28bab8 |
|
|
|
28bab8 |
free(f->fullpath);
|
|
|
28bab8 |
free(f->filename);
|
|
|
28bab8 |
@@ -39,10 +39,14 @@ void free_file_obj(file_obj_t *f)
|
|
|
28bab8 |
|
|
|
28bab8 |
const char *fo_get_fullpath(file_obj_t *fo)
|
|
|
28bab8 |
{
|
|
|
28bab8 |
+ g_return_val_if_fail(NULL != fo, NULL);
|
|
|
28bab8 |
+
|
|
|
28bab8 |
return fo->fullpath;
|
|
|
28bab8 |
}
|
|
|
28bab8 |
|
|
|
28bab8 |
const char *fo_get_filename(file_obj_t *fo)
|
|
|
28bab8 |
{
|
|
|
28bab8 |
+ g_return_val_if_fail(NULL != fo, NULL);
|
|
|
28bab8 |
+
|
|
|
28bab8 |
return fo->filename;
|
|
|
28bab8 |
-}
|
|
|
28bab8 |
\ No newline at end of file
|
|
|
28bab8 |
+}
|
|
|
28bab8 |
--
|
|
|
28bab8 |
2.21.0
|
|
|
28bab8 |
|