commit f79c908e0357ff9f76f565c1094899e7b75a98a3
author Dongjin Kim <dongjin_.kim@samsung.com> Tue Jun 01 17:20:48 2021 +0900
committer Brett Wilson <brettw@google.com> Tue Jun 01 17:42:18 2021 +0000
tree 6c1f702a5f24c11f2df517ca558b3d8c0aa0d2d4
parent 389557c113ceb630c67a842457073853a1747a13

[f2fs] Use fbl::MakeRefCounted when creating ref-counted objects

(Comment: https://fuchsia-review.googlesource.com/c/third_party/f2fs/+/536501/2/vnode.cc#74)

Test: fx test fs-tests
Failed tests: RwFullDiskTest.PartialWriteSucceedsForFullDisk/F2fs

Change-Id: I9d384cd348e22b9c0af89ebb0956901d5d8ca927
Reviewed-on: https://fuchsia-review.googlesource.com/c/third_party/f2fs/+/536743
Reviewed-by: Brett Wilson <brettw@google.com>

vnode.cc

diff --git a/vnode.cc b/vnode.cc
index 75f12a1..5761d16 100644
--- a/vnode.cc
+++ b/vnode.cc
@@ -41,9 +41,9 @@
   CheckNidRange(&fs->GetSbInfo(), ino);
 
   if (S_ISDIR(mode)) {
-    *out = fbl::AdoptRef(new Dir(fs, ino));
+    *out = fbl::MakeRefCounted<Dir>(fs, ino);
   } else {
-    *out = fbl::AdoptRef(new File(fs, ino));
+    *out = fbl::MakeRefCounted<File>(fs, ino);
   }
 
   VnodeF2fs *vnode = out->get();
@@ -71,7 +71,7 @@
   Node *rn;
 
   if (ino == NodeIno(&fs->GetSbInfo()) || ino == MetaIno(&fs->GetSbInfo())) {
-    *out = fbl::AdoptRef(new VnodeF2fs(fs, ino));
+    *out = fbl::MakeRefCounted<VnodeF2fs>(fs, ino);
     return;
   }
 
@@ -87,9 +87,9 @@
 
   // [sukka] need to check result?
   if (S_ISDIR(ri->i_mode)) {
-    *out = fbl::AdoptRef(new Dir(fs, ino));
+    *out = fbl::MakeRefCounted<Dir>(fs, ino);
   } else {
-    *out = fbl::AdoptRef(new File(fs, ino));
+    *out = fbl::MakeRefCounted<File>(fs, ino);
   }
 
   memcpy(&(*out)->inode_, ri, sizeof(Inode));