Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 201951
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Sven Wegener <swegener@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Giacomo Perale <ghepeu@virgilio.it>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
pinentry-0.7.4-grab.patch pinentry-0.7.4-grab.patch patch Alon Bar-Lev (RETIRED) 2007-12-21 23:32 0000 717 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 201951 depends on: Show dependency tree
Bug 201951 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2007-12-11 14:37 0000 
The patch pinentry-0.7.2-grab.patch works reliably with pinentry 0.7.3 and
metacity but breaks pinentry when using compiz 0.6.2. 
The passphrase is accepted when I use the [Enter] key but if I click on the OK
button pinentry crashes.
Commenting out the relevant epatch line from the ebuild and rebulding pinentry
fixes this problem.

Reproducible: Always

------- Comment #1 From Giacomo Perale 2007-12-20 23:19:13 0000 ------- 
The same's true for pinentry 0.7.4.

------- Comment #2 From Alon Bar-Lev (RETIRED) 2007-12-21 06:47:38 0000 ------- 
Any clue what the difference is?

------- Comment #3 From Alon Bar-Lev (RETIRED) 2007-12-21 23:29:30 0000 ------- 
This is following bug#165493.

I am no gtk+ developer... I tried to play with these events...

map-event is issued too early for some users... So grab does not work.
expose-event is issued too many times during session, so it may cause the
behavior we see here.

I tried to add gdk_event_get_graphics_expose() at the map-expose event handler,
but then it stacked.

I tried to see if there is another event happening after map-event, and found
focus-in-event that already used if nograb is used.

I don't really understand the function of the map-event here...

------- Comment #4 From Alon Bar-Lev (RETIRED) 2007-12-21 23:32:29 0000 ------- 
Created an attachment (id=139069) [details]
pinentry-0.7.4-grab.patch

I think this patch is "good enough", it handles focus-in/focus-out-event and
grab the keyboard if requested.

The extra protection of early grab set is dropped... But if it works, I think
it will do.

Can you all check if it works?

------- Comment #5 From Maurice van der Pot 2007-12-22 01:07:03 0000 ------- 
This patch works perfectly for me.

It turns out that with the previous patch pinentry crashed for me as well when
clicking OK. I only tested with the enter key before. 

I'm not running compiz though and disabling the composite extension did not
make a difference.

------- Comment #6 From Alon Bar-Lev (RETIRED) 2007-12-22 05:21:16 0000 ------- 
OK.
Closing this.
If someone has a better solution please reopen.
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug