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

"Akseli Lahtinen" <[email protected]>
Newsgroups gmane.comp.kde.devel.kfm
Message-ID <[email protected]/>
https://bugs.kde.org/show_bug.cgi?id=505215

Akseli Lahtinen <[email protected]> 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 <[email protected]> ---
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 <[email protected]>

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.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.