By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1438 - KArm: memory leak in idle detector
Summary: KArm: memory leak in idle detector
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdepim (show other bugs)
Version: 3.5.13.2 [Trinity]
Hardware: All All
: P5 major
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2013-04-07 18:27 CDT by Slávek Banko
Modified: 2013-04-08 11:22 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Fix memory leak in karm idle detector (707 bytes, patch)
2013-04-07 18:27 CDT, Slávek Banko 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Slávek Banko 2013-04-07 18:27:47 CDT 
Created attachment 1142 [details]
Fix memory leak in karm idle detector

I tried to find out why the idle detection in KArm ignores keyboard input and monitor only the movement of the mouse (for now no conclusive result).

During this time, I noticed that in the code is probably memory leak. For each idle timer test is called:

_mit_info XScreenSaverAllocInfo = ();

Nowhere is the test if memory for information has already allocated and nowhere is release of this memory. Please can someone confirm my findings? Proposed patch attached.
Comment 1 Timothy Pearson 2013-04-07 22:50:38 CDT 
Looks like a simple oversight of the original developers to me; the patch looks good and you should push it.

Tim
Comment 2 Slávek Banko 2013-04-08 11:22:08 CDT 
Fixed in GIT hash c4b86c6c and also in v3.5.13-sru branch.