Summary: | dolphin file descriptor leak when showing /tmp | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | alanas.00 |
Component: | view-engine: general | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | akselmo, kfm-devel |
Priority: | NOR | ||
Version First Reported In: | git-master | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/system/dolphin/-/commit/d19cc92c36fca8fb4cc43089fc64c7a8cce24284 | Version Fixed In: | |
Sentry Crash Report: | |||
Attachments: | backtrace of openat syscall that returned 42 (one of file descriptors that was never closed) |
Description
alanas.00
2025-06-04 18:27:40 UTC
Can confirm, will investigate Operating System: Fedora Linux 42 KDE Plasma Version: 6.4.80 KDE Frameworks Version: 6.15.0 Qt Version: 6.9.0 Kernel Version: 6.15.0-61.fc43.x86_64 (64-bit) Graphics Platform: Wayland Processors: 12 × AMD Ryzen 5 3600 6-Core Processor Memory: 16 GiB of RAM (15.5 GiB usable) Graphics Processor: AMD Radeon RX 6600 A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/983 Git commit 04e493d78cdf46e64562fe8a302426b1fd8c47df by Akseli Lahtinen. Committed on 09/06/2025 at 12:55. Pushed by akselmo into branch 'master'. viewproperties: Fix leaking file descriptors File descriptors would keep leaking here since tempFile never gets deleted. This would be especially noticeable when browsing /tmp/ folder. This patch makes the QTemporaryFile an unique_ptr, so it gets deleted when it's out of scope. This also causes the files to be handled accordingly. M +6 -19 src/views/viewproperties.cpp https://invent.kde.org/system/dolphin/-/commit/04e493d78cdf46e64562fe8a302426b1fd8c47df Git commit d19cc92c36fca8fb4cc43089fc64c7a8cce24284 by Akseli Lahtinen. Committed on 09/06/2025 at 14:42. Pushed by meven into branch 'release/25.04'. viewproperties: Fix leaking file descriptors File descriptors would keep leaking here since tempFile never gets deleted. This would be especially noticeable when browsing /tmp/ folder. This patch makes the QTemporaryFile an unique_ptr, so it gets deleted when it's out of scope. This also causes the files to be handled accordingly. (cherry picked from commit 04e493d78cdf46e64562fe8a302426b1fd8c47df) Co-authored-by: Akseli Lahtinen <akselmo@akselmo.dev> M +6 -19 src/views/viewproperties.cpp https://invent.kde.org/system/dolphin/-/commit/d19cc92c36fca8fb4cc43089fc64c7a8cce24284 |