Review Request 125969: kinit: fix Coverity issues + small optimization

Nick Shaforostoff shafff at ukr.net
Thu Nov 5 22:16:23 UTC 2015


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

Review request for KDE Frameworks and Alex Merry.


Repository: kinit


Description
-------

this patch fixes two coverity issues ranked 'outstanding':
https://scan5.coverity.com/reports.htm#v39099/p10103/fileInstanceId=82663767&defectInstanceId=24554334&mergedDefectId=258481
https://scan5.coverity.com/reports.htm#v39099/p10103/fileInstanceId=82663767&defectInstanceId=24557588&mergedDefectId=258474

and also does small string-related optimization by eliminating redundant mallocs done by QByteArray ctor


Diffs
-----

  src/kdeinit/kinit.cpp 9e775b6 

Diff: https://git.reviewboard.kde.org/r/125969/diff/


Testing
-------

compiles fine


Thanks,

Nick Shaforostoff

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151105/3a35a05f/attachment.html>


More information about the Kde-frameworks-devel mailing list