[PATCH] tools/gpgconf.c (main): Initialize libgcrypt.

Werner Koch wk at gnupg.org
Thu Aug 11 13:03:55 CEST 2016


On Wed, 10 Aug 2016 14:36, justus at g10code.com said:

> 1/ If the order is important, I don't like that, and we should fix
> that.  That sounds like the code is trying to be clever with the
> preprocessor.

config.h must always be the first header becuase it controls features
provided by the system headers.  The latter SHOULD prepend application
specific headers to ease debugging.

[ some magical way that the error source is set that your change ..]
> 2/ Likewise.  It might be convenient, but it sounds too magical to be
> good.  Explicit is better than implicit.

The tricky part is the GPG_ERR_SOURCE which has a default unless set in
another way.  In general not a problem but sometimes things get messed
up.

[i18n.h being included in common/init.c]
> 3/ I don't understand that point at all.

Pretty easy: This requires linking to libintl even for tools which have
no need for it.

> 4/ Is that actually a problem?  I just checked, none of the gnupg
> components on my Debian/Linux system are suid root, so that could only

On Linux this is not a problem as long as the amount of mlocked memory
is small enough.  On other OSes you may want to suid the application to
take advantage of mlocked memory.

> I believe that the change we merged looks very reasonable (hence I
> merged it), and such a change should not be able to have subtle
> side-effects like the ones Werner fears.  If there are weird implicit

The gcry_check_version is important and chould be done explicit.


Shalom-Salam,

   Werner

-- 
Die Gedanken sind frei.  Ausnahmen regelt ein Bundesgesetz.
 /* Join us at OpenPGP.conf  <https://openpgp-conf.org> */




More information about the Gnupg-devel mailing list