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 2222 - kdesktop_lock remained active after awakening
Summary: kdesktop_lock remained active after awakening
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P1 critical
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: 2014
  Show dependency tree / graph
 
Reported: 2014-12-01 16:38 CST by Slávek Banko
Modified: 2014-12-02 21:10 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version: 7cf0c0b4
Application Name: kdesktop_lock

Attachments
kdesktop and kdesktop_lock backtrace (2.88 KB, text/plain)
2014-12-01 16:38 CST, Slávek Banko 		Details
Forcibly terminate kdesktop_lock after unlock (615 bytes, patch)
2014-12-01 18:27 CST, Timothy Pearson 		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 2014-12-01 16:38:07 CST 
Created attachment 2368 [details]
kdesktop and kdesktop_lock backtrace

For the second time in a short time I have encountered a situation that kdesktop_lock remained active => input was still grabbed, even though the screen was active - saver is not running.
Comment 1 Timothy Pearson 2014-12-01 18:27:19 CST 
Created attachment 2370 [details]
Forcibly terminate kdesktop_lock after unlock

Does this patch fix the issue?  It reverts a small part of GIT hash 2f7d50c2, restoring the R14 RC1 kdesktop_lock behavior at the expense of reintroducing a small bug where DCOP hangs and the screen locker does not engage.
Comment 2 Timothy Pearson 2014-12-02 10:28:06 CST 
Do you mind testing the patch?  I need to know if it should be applied for R14 final.

Thanks!
Comment 3 Slávek Banko 2014-12-02 10:32:38 CST 
(In reply to Timothy Pearson from comment #2)
> Do you mind testing the patch?  I need to know if it should be applied for
> R14 final.
> 
> Thanks!

Yes, I built tdebase with this patch, installed on my test machine, and not once I observed the previous problem. It looks good.
Comment 4 Timothy Pearson 2014-12-02 11:12:23 CST 
(In reply to Slávek Banko from comment #3)
> (In reply to Timothy Pearson from comment #2)
> > Do you mind testing the patch?  I need to know if it should be applied for
> > R14 final.
> > 
> > Thanks!
> 
> Yes, I built tdebase with this patch, installed on my test machine, and not
> once I observed the previous problem. It looks good.

OK, thanks!  I'll apply it for the R14 final release then.
Comment 5 Timothy Pearson 2014-12-02 21:10:18 CST 
Pushed to GIT in hash aaa719d.