Bug 375875

Summary: Changing password does not work
Product: [Applications] skrooge Reporter: Christoph Vogtländer <kde>
Component: generalAssignee: Stephane MANKOWSKI <stephane>
Status: RESOLVED FIXED    
Severity: normal CC: stephane
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: 25.1.0   
Platform: Kubuntu   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Christoph Vogtländer 2017-02-02 00:04:07 UTC
After upgrading to skrooge 2.7.0 I get a warning that a security hole has been detected in an earlier version and upgrading the password is strongly advised.

Trying to change the password will result in the following error (sorry, some messages in German even if starting skrooge with LANG=en):
 [ERR-5]: Failed to change password.
 [ERR-5]: Creation failed. The object already exists.
 [ERR-10019]: Das Objekt kann nicht eingefügt werden (doctransaction wird von doctransactionmsg verwendet).
 Name der Einschränkung: fki_doctransactionmsg_doctransaction_rd_doctransaction_id_id Der Datensatz konnte nicht abgeholt werden
 [ERR-10019]: INSERT INTO doctransactionmsg ('t_message','rd_doctransaction_id','t_type') VALUES ('The document password has been modified.','22532','P')

Changing something and trying to save will result in:
 [ERR-5]: Cannot save file
 [ERR-10026]: file is encrypted or is not a database Der Datensatz konnte nicht abgeholt werden
 [ERR-10026]: update doctransaction set t_savestep='Y' where id in (select A.id from doctransaction A where NOT EXISTS(select 1 from doctransaction B where B.i_parent=A.id) and A.t_mode='U')
Comment 1 Stephane MANKOWSKI 2017-02-02 09:34:52 UTC
Hi,

Could you generate traces by doing this from a console?
1-export SKGTRACE=10
2-export SKGTRACESQL=1
3-skrooge > traces.txt
4-play your scenario
5-send me the traces BY EMAIL (my email is in the about of skrooge)

Regards.
Stéphane
Comment 2 Stephane MANKOWSKI 2017-02-03 22:35:26 UTC
Git commit aad18a1acf998d1c285484371f467f5a05b57ee1 by Stephane Mankowski.
Committed on 03/02/2017 at 22:32.
Pushed by smankowski into branch 'master'.

Changing password does not work

M  +1    -0    CHANGELOG
M  +1    -2    plugins/generic/skg_file/skgfileplugin.cpp
M  +20   -9    skgbasemodeler/skgdocument.cpp
M  +1    -1    skgbasemodeler/skgdocument.h
M  +2    -2    tests/skgbankmodelertest/skgtestimportafb120.cpp
M  +2    -2    tests/skgbankmodelertest/skgtestimportskg.cpp
M  +3    -4    tests/skgbasemodelertest/skgtestfile.cpp

https://commits.kde.org/skrooge/aad18a1acf998d1c285484371f467f5a05b57ee1