summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Kling <awesomekling@gmail.com>2019-11-04 00:21:08 +0100
committerAndreas Kling <awesomekling@gmail.com>2019-11-04 00:21:08 +0100
commite3dda4e87b39eb142d85dd520af3c772b793a2fb (patch)
tree8bce1db79f0cc572de496ea431adf80d89104080
parent9b2dc362292293394d212e0d7b0ac86579d077e9 (diff)
downloadserenity-e3dda4e87b39eb142d85dd520af3c772b793a2fb.zip
Kernel: Fix weird Region constructor that took nullable RefPtr<Inode>
It's never valid to construct a Region with a null Inode pointer using this constructor, so just take a NonnullRefPtr<Inode> instead.
-rw-r--r--Kernel/VM/Region.cpp2
-rw-r--r--Kernel/VM/Region.h2
2 files changed, 2 insertions, 2 deletions
diff --git a/Kernel/VM/Region.cpp b/Kernel/VM/Region.cpp
index 682f28f55c..bb2173cf73 100644
--- a/Kernel/VM/Region.cpp
+++ b/Kernel/VM/Region.cpp
@@ -17,7 +17,7 @@ Region::Region(const Range& range, const String& name, u8 access)
MM.register_region(*this);
}
-Region::Region(const Range& range, RefPtr<Inode>&& inode, const String& name, u8 access)
+Region::Region(const Range& range, NonnullRefPtr<Inode> inode, const String& name, u8 access)
: m_range(range)
, m_vmobject(InodeVMObject::create_with_inode(*inode))
, m_name(name)
diff --git a/Kernel/VM/Region.h b/Kernel/VM/Region.h
index 104661bfad..1d9ce7f5d3 100644
--- a/Kernel/VM/Region.h
+++ b/Kernel/VM/Region.h
@@ -131,7 +131,7 @@ public:
// NOTE: These are public so we can make<> them.
Region(const Range&, const String&, u8 access);
Region(const Range&, NonnullRefPtr<VMObject>, size_t offset_in_vmo, const String&, u8 access);
- Region(const Range&, RefPtr<Inode>&&, const String&, u8 access);
+ Region(const Range&, NonnullRefPtr<Inode>, const String&, u8 access);
private:
Bitmap& ensure_cow_map() const;