bug-gnulib@gnu.org mirror (unofficial)
 help / color / mirror / Atom feed
* crypto/gc-md2: add comment
@ 2019-08-24 13:02 Bruno Haible
  0 siblings, 0 replies; only message in thread
From: Bruno Haible @ 2019-08-24 13:02 UTC (permalink / raw)
  To: bug-gnulib

The gc-libgcrypt.c file always takes the md2 implementation from gnulib.
It should get it from libgcrypt, when this library is available, like for
md4, md5, sha1.

How to reproduce:
$ ./gnulib-tool --create-testdir --dir=../testdir1 --single-configure \
  crypto/gc-md2 crypto/gc-md4 crypto/gc-md5 crypto/gc-rijndael crypto/gc-sha1
$ cd ../testdir1
$ ./configure --with-libgcrypt
$ make
$ nm gltests/test-gc-md4 | grep md4
00000000004017e0 T gc_md4
$ nm gltests/test-gc-md5 | grep md5
00000000004018d0 T gc_md5
$ nm gltests/test-gc-sha1 | grep sha1
00000000004019b0 T gc_sha1
$ nm gltests/test-gc-md2 | grep md2
00000000004017e0 T gc_md2
0000000000401e00 T md2_buffer
0000000000401aa0 t md2_compress
0000000000401b90 T md2_finish_ctx
0000000000401b10 T md2_init_ctx
0000000000401e60 T md2_process_block
0000000000401c30 T md2_process_bytes
0000000000401b70 T md2_read_ctx
0000000000401cf0 T md2_stream

The reason is that libgcrypt declares GCRY_MD_MD2 but does not actually
implement the MD2 algorithm. This is worth a comment.


2019-08-24  Bruno Haible  <bruno@clisp.org>

	crypto/gc-md2: Add comment.
	* lib/gc-libgcrypt.c: Add comment.

diff --git a/lib/gc-libgcrypt.c b/lib/gc-libgcrypt.c
index f92fabc..6c95712 100644
--- a/lib/gc-libgcrypt.c
+++ b/lib/gc-libgcrypt.c
@@ -29,6 +29,8 @@
 /* Get libgcrypt API. */
 #include <gcrypt.h>
 #if GNULIB_GC_MD2
+/* libgcrypt declares GCRY_MD_MD2 but does not actually implement the
+   MD2 algorithm.  Therefore take the implementation from gnulib.  */
 # include "md2.h"
 #endif
 



^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2019-08-24 13:02 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-24 13:02 crypto/gc-md2: add comment Bruno Haible

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).