Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: GnuPG: gpa

pinentry-gtk-2 SIGABRT when unable to grab keyboard

 

 

GnuPG gpa RSS feed   Index | Next | Previous | View Threaded


sochotnicky at redhat

Apr 26, 2010, 5:31 AM

Post #1 of 5 (2483 views)
Permalink
pinentry-gtk-2 SIGABRT when unable to grab keyboard

Hi,


we received quite a few bugreports about pinentry-gtk-2 abort-ing when
it's unable to grab keyboard. See:
https://bugzilla.redhat.com/show_bug.cgi?id=554884


I can see why we should be careful when unable to grab keyboard
(passphrase sniffing, accidental input of passphrase to wrong window
etc). But I am also wondering if we can make this better, because let's
face it...crashing on users is not OK under normal circumstances. And
pinentry can be used in situations when it cannot grab keyboard
immediately but has to wait a bit...

Code causing behaviour in question:
if (gdk_keyboard_grab (win->window, FALSE, gdk_event_get_time (event)))
g_error ("could not grab keyboard");


Abort happens in function grab_keyboard and can quite easily be
reproduced by doing following (from bug #554884):
1. execute echo getpin | pinentry-gtk-2
2. change to another workspace using mouse (keyboard is grabbed)
3. change back to previous workspace using keyboard shortcut


I believe changing workspace should not cause pinentry-gtk-2 to abort
because this could be used for example to change to workspace with
password manager program. There are also other ways to trigger this
behaviour.

I can think of either gracefully exiting pinentry-gtk-2, or even
better...provide some sort of visual clue that there is a problem with
grabbing keyboard and that user should not start typing their password
yet. Myabe I missed some aspect of the problem entirely, so feel free to
enlighten me.



Regards,

--
Stanislav Ochotnicky <sochotnicky [at] redhat>
Associate Software Engineer - Base Operating Systems Brno

Mobile: +420 775 633 759
Red Hat Inc. http://cz.redhat.com
Attachments: signature.asc (0.19 KB)


wk at gnupg

Apr 26, 2010, 9:08 AM

Post #2 of 5 (2382 views)
Permalink
Re: pinentry-gtk-2 SIGABRT when unable to grab keyboard [In reply to]

On Mon, 26 Apr 2010 14:31, sochotnicky [at] redhat said:

> Code causing behaviour in question:
> if (gdk_keyboard_grab (win->window, FALSE, gdk_event_get_time (event)))
> g_error ("could not grab keyboard");

The g_error causes the abort. This is probably because the function
name g_error does not indicate that this is fatal. It would have been
much better if the gtk folks would have named this g_fatal or g_die. I
stepped into this trap several times in the past.

Anyway I changed it so that the output will be

OK Your orders please

** (pinentry-gtk-2:20766): CRITICAL **: could not grab keyboard
ERR 83886179 canceled

> I believe changing workspace should not cause pinentry-gtk-2 to abort
> because this could be used for example to change to workspace with
> password manager program. There are also other ways to trigger this

It is actually intended that you can't use a password manager -
gpg-agent is the password manager! If it would be possible to cut and
paste between windows other programs could do this as well; we try to
make this harder by grabbing the keyboard. Put "no-grab" into
gpg-agent.conf if you don't like this feature.


Salam-Shalom,

Werner

--
Die Gedanken sind frei. Ausnahmen regelt ein Bundesgesetz.


_______________________________________________
Gpa-dev mailing list
Gpa-dev [at] gnupg
http://lists.gnupg.org/mailman/listinfo/gpa-dev


marcus.brinkmann at ruhr-uni-bochum

Apr 26, 2010, 12:28 PM

Post #3 of 5 (2373 views)
Permalink
Re: pinentry-gtk-2 SIGABRT when unable to grab keyboard [In reply to]

On 04/26/2010 06:08 PM, Werner Koch wrote:
> It is actually intended that you can't use a password manager -
> gpg-agent is the password manager!

gpg-agent can cache passphrases, but it does not commit them to persistent
storage, and also does not provide a wallet-like functionality. If it is
intended to replace those, we should add these features.

Thanks,
Marcus

_______________________________________________
Gpa-dev mailing list
Gpa-dev [at] gnupg
http://lists.gnupg.org/mailman/listinfo/gpa-dev


kp2010 at koospol

Apr 27, 2010, 11:24 AM

Post #4 of 5 (2369 views)
Permalink
Re: pinentry-gtk-2 SIGABRT when unable to grab keyboard [In reply to]

On maandag 26 april 2010, Werner Koch wrote:

> It is actually intended that you can't use a password manager -
> gpg-agent is the password manager! If it would be possible to cut and
> paste between windows other programs could do this as well; we try to
> make this harder by grabbing the keyboard. Put "no-grab" into
> gpg-agent.conf if you don't like this feature.

I miss this dearly. I'd like to use strong passwords generated by KeepassX,
but the inability to copy/paste makes this impossible. Hence I use one easy to
remember password for all my keys. If there is any way around, I'd want to be
the first to know :-)

Cheers,
Koos

--
http://www.malarianomore.org
For $10 a mosquito net + distribution + education + monitoring + evaluation.


_______________________________________________
Gpa-dev mailing list
Gpa-dev [at] gnupg
http://lists.gnupg.org/mailman/listinfo/gpa-dev


wk at gnupg

May 3, 2010, 6:53 AM

Post #5 of 5 (2362 views)
Permalink
Re: pinentry-gtk-2 SIGABRT when unable to grab keyboard [In reply to]

On Mon, 26 Apr 2010 21:28, marcus.brinkmann [at] ruhr-uni-bochum said:

> gpg-agent can cache passphrases, but it does not commit them to persistent
> storage, and also does not provide a wallet-like functionality. If it is
> intended to replace those, we should add these features.

Good point.


Salam-Shalom,

Werner

--
Die Gedanken sind frei. Ausnahmen regelt ein Bundesgesetz.


_______________________________________________
Gpa-dev mailing list
Gpa-dev [at] gnupg
http://lists.gnupg.org/mailman/listinfo/gpa-dev

GnuPG gpa RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.