[Debian-ha-commits] [ocfs2-tools] 13/58: libocfs2: fix risk of using uninitialized variables

Valentin Vidic vvidic-guest at moszumanska.debian.org
Tue Jun 20 13:40:40 UTC 2017


This is an automated email from the git hooks/post-receive script.

vvidic-guest pushed a commit to branch upstream
in repository ocfs2-tools.

commit 72a422fb96ed43bd6a2b5ab35630e8b9b540f383
Author: Joseph Qi <joseph.qi at huawei.com>
Date:   Tue Jul 12 21:26:57 2016 +0800

    libocfs2: fix risk of using uninitialized variables
    
    suballoc_bit in ocfs2_new_system_inode and p_blkno in
    ocfs2_allocate_unwritten_extents may be used without initialized, so
    fix them.
    
    Signed-off-by: Joseph Qi <joseph.qi at huawei.com>
---
 libocfs2/alloc.c       | 3 ++-
 libocfs2/extend_file.c | 5 ++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/libocfs2/alloc.c b/libocfs2/alloc.c
index 1cd21c6..258d4e5 100644
--- a/libocfs2/alloc.c
+++ b/libocfs2/alloc.c
@@ -306,7 +306,8 @@ errcode_t ocfs2_new_system_inode(ocfs2_filesys *fs, uint64_t *ino,
 						&gd_blkno, &suballoc_bit, ino);
 		if (ret)
 			goto out;
-	}
+	} else if (ret)
+		goto out;
 
 	memset(buf, 0, fs->fs_blocksize);
 	di = (struct ocfs2_dinode *)buf;
diff --git a/libocfs2/extend_file.c b/libocfs2/extend_file.c
index fbb770a..1038933 100644
--- a/libocfs2/extend_file.c
+++ b/libocfs2/extend_file.c
@@ -169,7 +169,7 @@ errcode_t ocfs2_allocate_unwritten_extents(ocfs2_filesys *fs, uint64_t ino,
 {
 	errcode_t ret = 0;
 	uint32_t n_clusters = 0, cpos;
-	uint64_t p_blkno, v_blkno, v_end, contig_blocks, wanted_blocks;
+	uint64_t p_blkno = 0, v_blkno, v_end, contig_blocks, wanted_blocks;
 	ocfs2_cached_inode *ci = NULL;
 
 	if (!(fs->fs_flags & OCFS2_FLAG_RW))
@@ -198,6 +198,9 @@ errcode_t ocfs2_allocate_unwritten_extents(ocfs2_filesys *fs, uint64_t ino,
 		ret = ocfs2_extent_map_get_blocks(ci, v_blkno, 1,
 						  &p_blkno, &contig_blocks,
 						  NULL);
+		if (ret)
+			continue;
+
 		if (p_blkno) {
 			v_blkno += contig_blocks;
 			continue;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-ha/ocfs2-tools.git



More information about the Debian-HA-Commits mailing list