[dolphin] [Bug 505215] dolphin file descriptor leak when showing /tmp

[Akseli Lahtinen]

https://bugs.kde.org/show_bug.cgi?id=505215

Akseli Lahtinen <akselmo at akselmo.dev> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/syst |https://invent.kde.org/syst
                   |em/dolphin/-/commit/04e493d |em/dolphin/-/commit/d19cc92
                   |78cdf46e64562fe8a302426b1fd |c36fca8fb4cc43089fc64c7a8cc
                   |8c47df                      |e24284

--- Comment #4 from Akseli Lahtinen <akselmo at akselmo.dev> ---
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 at akselmo.dev>

M  +6    -19   src/views/viewproperties.cpp

https://invent.kde.org/system/dolphin/-/commit/d19cc92c36fca8fb4cc43089fc64c7a8cce24284

-- 
You are receiving this mail because:
You are on the CC list for the bug.