|
Bugzilla – Full Text Bug Listing |
| Summary: | kbilliards-0.8.6-3: local variable used before set | ||
|---|---|---|---|
| Product: | [openSUSE] SUSE LINUX 10.0 | Reporter: | David Binderman <dcb314> |
| Component: | Basesystem | Assignee: | Michael Skibbe <mskibbe> |
| Status: | RESOLVED FIXED | QA Contact: | E-mail List <qa-bugs> |
| Severity: | Minor | ||
| Priority: | P5 - None | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | SuSE Linux 10.1 | ||
| Whiteboard: | |||
| Found By: | Other | Services Priority: | |
| Business Priority: | Blocker: | --- | |
| Marketing QA Status: | --- | IT Deployment: | --- |
Why is this a security risk? (In reply to comment #1) > Why is this a security risk? I made no mention of security in my original bug report, but I'll have a go anyway. I can say that the function timerEvent obviously gets given random data. That may be a security risk, beyond the usual programming error. I'll reassign this to Michael. Can take some time until it gets fixed, he's absent at the moment. i fixed this. |
I just tried to compile package kbilliards-0.8.6-3 with a prerelease version 4.2 of the GNU C compiler. It said ./src/soundmodule.cpp:77: warning: "e" is used uninitialized in this function The source code is QTimerEvent * e; // e(QEvent::Timer) ; KURL pathfile(path_sound + file ); if ( musicEngine->load(pathfile) ) timerEvent( e ); // ( &e ); Suggest initialise local variable "e" before first use.