diff options
| author | Andreas Kling <kling@serenityos.org> | 2023-03-06 19:29:25 +0100 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2023-03-07 00:30:12 +0100 |
| commit | d1371d66f79d7482b4fe1acd58803c0a5db734f4 (patch) | |
| tree | 98ead3246508f60971ceebf8478f0f22ebb7d98c /Kernel/TTY/PTYMultiplexer.h | |
| parent | 36b0ecfe9e28c00f12cde7add7330c4ec8cf3339 (diff) | |
| download | serenity-d1371d66f79d7482b4fe1acd58803c0a5db734f4.zip | |
Kernel: Use non-locking {Nonnull,}RefPtr for OpenFileDescription
This patch switches away from {Nonnull,}LockRefPtr to the non-locking
smart pointers throughout the kernel.
I've looked at the handful of places where these were being persisted
and I don't see any race situations.
Note that the process file descriptor table (Process::m_fds) was already
guarded via MutexProtected.
Diffstat (limited to 'Kernel/TTY/PTYMultiplexer.h')
| -rw-r--r-- | Kernel/TTY/PTYMultiplexer.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Kernel/TTY/PTYMultiplexer.h b/Kernel/TTY/PTYMultiplexer.h index c9da6ddbe0..441a7169df 100644 --- a/Kernel/TTY/PTYMultiplexer.h +++ b/Kernel/TTY/PTYMultiplexer.h @@ -22,7 +22,7 @@ public: static PTYMultiplexer& the(); // ^CharacterDevice - virtual ErrorOr<NonnullLockRefPtr<OpenFileDescription>> open(int options) override; + virtual ErrorOr<NonnullRefPtr<OpenFileDescription>> open(int options) override; virtual ErrorOr<size_t> read(OpenFileDescription&, u64, UserOrKernelBuffer&, size_t) override { return 0; } virtual ErrorOr<size_t> write(OpenFileDescription&, u64, UserOrKernelBuffer const&, size_t) override { return 0; } virtual bool can_read(OpenFileDescription const&, u64) const override { return true; } |