Bug 358469 - kwalletmanager5 resets Auto Allow setting
Summary: kwalletmanager5 resets Auto Allow setting
Status: RESOLVED FIXED
Alias: None
Product: kwalletmanager
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Valentin Rusu
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-24 10:43 UTC by Filippe
Modified: 2016-08-11 13:05 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Filippe 2016-01-24 10:43:34 UTC
If "Prompt on Open" in kwalletrc is set to true, opening (even without changing any settings) kwalletmanager5 causes deletion of [Auto Allow] section.

Reproducible: Always

Steps to Reproduce:
1. Check "Prompt when application acceses a wallet" setting.
2. Select "always allow" when prompted to give access to wallet for a program.
3. Open kwalletmanager.
4. Re-login and launch that program again.

Actual Results:  
Access request appeares again.
Comment 1 Burkhard Lück 2016-08-11 13:05:41 UTC
Git commit 0a7279fc0ebab9c8dbc94c6035751d0500aa342d by Burkhard Lück.
Committed on 11/08/2016 at 13:04.
Pushed by lueck into branch 'Applications/16.08'.

Fix wrong path to existing wallet files

the wallet files *.kwl + *.salt are located in a subdir "/kwalletd/" in QStandardPaths::GenericDataLocation
see
kwalletmanager.cpp:390:    const QString path = QStandardPaths::writableLocation(QStandardPaths::GenericDataLocation) + "/kwalletd/";
kwalletmanager.cpp:420:    const QString destinationDir = QStandardPaths::writableLocation(QStandardPaths::GenericDataLocation) + "/kwalletd/";

But the code in konfigurator.cpp used QStandardPaths::DataLocation) + "/kwallet/" therefore existing wallet files were not found
and the config entries "Auto Allow" and "Auto Deny" were always deleted
With the patch the config entries "Auto Allow" and "Auto Deny" are preserved in kwalletrc
REVIEW:128354
Related: bug 363532

M  +2    -2    src/konfigurator/konfigurator.cpp

http://commits.kde.org/kwallet/0a7279fc0ebab9c8dbc94c6035751d0500aa342d