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

Nick Shaforostoff shafff at ukr.net
Fri Nov 6 16:00:30 UTC 2015


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

(Updated Nov. 6, 2015, 4 p.m.)


Review request for KDE Frameworks and Alex Merry.


Changes
-------

now displayEnvVarName_c() returns const char* and displayEnvVarName() is wrapping it into a QByteArray


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 (updated)
-----

  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/20151106/bb472988/attachment.html>


More information about the Kde-frameworks-devel mailing list