[kde-edu]: Review Request: Prevent KwordQuiz crash when opening CVS files that dont contain <TAB> characters

Aleks aletum at lavabit.com
Wed Jan 13 13:22:26 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2569/
-----------------------------------------------------------

Review request for KDE-Edu.


Summary
-------

After KEduVocCsvReader::readDoc returns, m_doc->identifierCount() is either zero (an empty file) or 1 (only one TAB as the very last character). In either cases a crash occurs later on when KWQTableModel::headerData processes a non-existent m_doc->identifier(0).
P.S. Parley is not affected 


Diffs
-----

  /trunk/KDE/kdeedu/kwordquiz/src/kwordquiz.cpp 1071905 

Diff: http://reviewboard.kde.org/r/2569/diff


Testing
-------

If a CSV file contains no TABs, then KEduVocCsvReader::readDoc returns with m_doc->identifierCount() either zero (an empty file) or 1 (just one TAB as the last character of the file),
In either case a crash occurs later on when KWQTableModel::headerData processes a non-existent m_doc->identifier(0). 
P.S. Parley is not affected


Thanks,

Aleks



More information about the kde-edu mailing list