Suspicous code in kdelibs-3.5.2 Part 2
Christoph Bartoschek
bartoschek at gmx.de
Wed Apr 5 20:44:35 BST 2006
I forgot to check the .cc files in the first run.
------------------------------------------------------------------
Misc problems:
------------------------------------------------------------------
- libkmid/player.cc:365, 366
What happens when ev->length == 1024? then the ?-condition is false
and pspev->text[1024] is overwritten. Off by one error.
- arts/kde/kiotestslow.cc:38
There is no argument for the %d parameter.
------------------------------------
Problems involving the NULL pointer:
------------------------------------
- libkmid/voiceman.cc:59
What happens when totalvoices == 1? Then nvoices == 1 and the for loop
in line 47 is never entered. Then LastVoice becomes ptr which is still
NULL from line 45 and line 59 crashes.
-----------------------------------------------------------------
Cases from switch statements that fall through in some cases but
do not have a fall through comment as in most such cases.
------------------------------------------------------------------
- kioslave/http/http.cc:1655
-----------------------------------------------------------------
Lines where boolean expressions are used in non-boolean contexts:
-----------------------------------------------------------------
- kioslave/http/http.cc:5890
- kioslave/http/http.cc:5884
More information about the kde-core-devel
mailing list