[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [kwalletmanager] [Bug 254198] Kwalletd locks when 2 applications try to open the same wallet, the se
From:       Valentin Rusu <kde () rusu ! info>
Date:       2013-09-01 20:29:25
Message-ID: bug-254198-17878-clLwzXFsGo () http ! bugs ! kde ! org/
[Download RAW message or body]

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

Valentin Rusu <kde@rusu.info> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/kde-
                   |                            |runtime/b0f9053ed8abff4ef97
                   |                            |3b10842f761422ee17f41

--- Comment #35 from Valentin Rusu <kde@rusu.info> ---
Git commit b0f9053ed8abff4ef973b10842f761422ee17f41 by Valentin Rusu.
Committed on 31/08/2013 at 23:28.
Pushed by vrusu into branch 'master'.

Fix the synchronous-mode wallet open logic

The wallet synchronous open requests now use qdbus delayed replies.
The execution path now returns to the main event loop instead of
a nested event loop. The wallet opening UI logic is correctly handled
no longer leading to a frozen kwalletd.

Beware that this commit should be used along with the corresponding
fix of the kdelibs/kdeui module. Failing to updating kdelibs lead to
an aparently similar freeze condition, as kdeui also used an internal
event loop, faking synchronous operation when making async kwalletd calls.

M  +0    -1    kwalletd/CMakeLists.txt
M  +53   -21   kwalletd/kwalletd.cpp
D  +0    -46   kwalletd/kwalletopenloop.cpp
D  +0    -50   kwalletd/kwalletopenloop.h
M  +1    -0    kwalletd/main.cpp

http://commits.kde.org/kde-runtime/b0f9053ed8abff4ef973b10842f761422ee17f41

-- 
You are receiving this mail because:
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic