mirrors/nextcloud-desktop
1
0
Fork 0
mirror of https://github.com/nextcloud/desktop.git synced 2024-10-25 05:45:42 +03:00
Code Issues Projects Releases Packages Wiki Activity

Do not use QSharedPointer to store FileProviderSocketControllers, use QPointer

Browse source 
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
This commit is contained in:
Claudio Cambra 2023-03-15 17:56:56 +01:00
parent d3c3181c0d
commit 9fff10fd5d
No known key found for this signature in database
GPG key ID: C839200C384636B0
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/gui/macOS/fileprovidersocketserver.cpp
View file

@ -71,7 +71,12 @@ void FileProviderSocketServer::slotNewConnection()
void FileProviderSocketServer::slotSocketDestroyed(const QLocalSocket * const socket)
{
_socketControllers.remove(socket);
const auto socketController = _socketControllers.take(socket);
if (socketController) {
const auto rawSocketControllerPtr = socketController.data();
delete rawSocketControllerPtr;
}
}
} // namespace Mac

2
src/gui/macOS/fileprovidersocketserver.h
View file

@ -22,7 +22,7 @@ namespace OCC {
namespace Mac {
class FileProviderSocketController;
using FileProviderSocketControllerPtr = QSharedPointer<FileProviderSocketController>;
using FileProviderSocketControllerPtr = QPointer<FileProviderSocketController>;
QString fileProviderSocketPath();