Bug 336422

Summary: Adding false friends causes Editor to crash
Product: [Applications] parley Reporter: Andreas <andxav>
Component: editorAssignee: parley bug tracker <parley-bugs-null>
Status: RESOLVED FIXED    
Severity: crash    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: http://commits.kde.org/parley/95a445a1f6ee62708f87588d6594dffb953117a9 Version Fixed In:
Attachments: example

Description Andreas 2014-06-19 08:18:04 UTC 
1. Parley -> Load Collection -> Editor
2. View -> False friends
3. Select 2 words
4. Make them false friends
5. Crash

Reproducible: Always
Comment 1 Andreas 2014-06-19 09:44:32 UTC 
Created attachment 87275 [details]
example
Comment 2 Andreas 2014-07-06 17:27:11 UTC 
Git commit 95a445a1f6ee62708f87588d6594dffb953117a9 by Andreas Xavier.
Committed on 06/07/2014 at 17:24.
Pushed by axavier into branch 'master'.

Fix null pointer reference in SynonymWidget::togglePair() by initializing pointer

In both cases (antonyms and falseFriends) the synonymWidget::togglePair() was referencing an uninitialized pointer m_doc to call setModified() on the  KEduVocDocument.  The quick fix was to store pointers to the antonym and falseFriend widgets and initialize m_doc.  A better long term fix would be to have the kedevocdoc manage its own state and determine when it is dirty on its own.
Related: bug 336419

REVIEW: 119107

M  +8    -6    src/editor/editor.cpp
M  +2    -1    src/editor/editor.h

http://commits.kde.org/parley/95a445a1f6ee62708f87588d6594dffb953117a9