Review Request 128128: Parley: Fix Coverty Scan ID 1342591 - Dereference before null check

[Andreas Cord-Landwehr]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128128/#review96301
-----------------------------------------------------------


Ship it!





src/editor/multiplechoicewidget.cpp (line 33)
<https://git.reviewboard.kde.org/r/128128/#comment65099>

    in an application we can also use nulltpr, since we do not have such strict compiler support requirements like KF5; and actually nullptr is already used in Parley code



src/editor/multiplechoicewidget.cpp (line 57)
<https://git.reviewboard.kde.org/r/128128/#comment65100>

    that looks awkward, at least uncommon to use references here; I wonder why there is a method setMultipleChoice(...) which is commented out... maybe a code improvement in another review? :)


- Andreas Cord-Landwehr


On Juni 8, 2016, 7:49 nachm., Hartmut Riesenbeck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128128/
> -----------------------------------------------------------
> 
> (Updated Juni 8, 2016, 7:49 nachm.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Repository: parley
> 
> 
> Description
> -------
> 
> The m_translation pointer member of MultipleChoiceWidget class is now
> inititalized in constructor and missing null pointer checks before
> usage were added.
> 
> 
> Diffs
> -----
> 
>   src/editor/multiplechoicewidget.h c1522e5c6cd630e538fba61886f9dbf1faecced2 
>   src/editor/multiplechoicewidget.cpp bcfc5118a7aa54da18413682fab93f5d7116e8dd 
> 
> Diff: https://git.reviewboard.kde.org/r/128128/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hartmut Riesenbeck
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20160608/77b924c9/attachment.html>