Review Request 129599: segfault when ruby was not installed

Leslie Zhai xiangzhai83 at gmail.com
Mon Dec 5 02:32:07 UTC 2016


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

(Updated 十二月 5, 2016, 10:32 a.m.)


Review request for KDevelop and Kevin Funk.


Changes
-------

Fix as Sven Brauch suggested


Repository: kdev-ruby


Description
-------

if ruby was not installed, ruby.readAllStandardOutput().split(' ')'s size is ZERO, so I simply added the check:

```
QByteArray allOutput = ruby.readAllStandardOutput();
if (allOutput.split(' ').size() > 1)
```


Diffs (updated)
-----

  languagesupport.cpp d618d7d 

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


Testing
-------

1. Not intalled ruby, kdevelop import redmin (a bug tracker written in ruby) will NOT segfault
2. Installed ruby, the same story


Thanks,

Leslie Zhai

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161205/dc0fc79c/attachment.html>


More information about the KDevelop-devel mailing list