[PATCH] Check the return value when starting gpg.
Ben Kibbey
bjk at luxsci.net
Wed Aug 12 12:45:18 CEST 2015
On Wed, Aug 12, 2015 at 08:14:00AM +0200, Werner Koch wrote:
> On Tue, 11 Aug 2015 03:26, bjk at luxsci.net said:
> > * src/engine-gpg.c (gpg_decrypt, gpg_delete, gpg_passwd): Check return
> > value of start().
>
> Thanks. Any special reason why you did not changed this for gpg_sign()
> too?
None other than I overlooked it. :)
> I wonder why Marcus checked the return value only on some invocations
> when he chnaged the code in 2003.
Not sure. But when stress testing I'm getting crashes in
_gpgme_io_select() so there may be a race condition which may be related
to the FIXME's in start(). Without returning an error from start() it
gets stuck in a _gpgme_io_select() loop.
--
Ben Kibbey
More information about the Gnupg-devel
mailing list