diff options
author | Andreas Kling <awesomekling@gmail.com> | 2019-11-04 00:21:08 +0100 |
---|---|---|
committer | Andreas Kling <awesomekling@gmail.com> | 2019-11-04 00:21:08 +0100 |
commit | e3dda4e87b39eb142d85dd520af3c772b793a2fb (patch) | |
tree | 8bce1db79f0cc572de496ea431adf80d89104080 | |
parent | 9b2dc362292293394d212e0d7b0ac86579d077e9 (diff) | |
download | serenity-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.cpp | 2 | ||||
-rw-r--r-- | Kernel/VM/Region.h | 2 |
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; |