[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [gnutls-dev] [PATCH] Fixing OpenPGP keyring import (again)

From: Ludovic Courtès
Subject: Re: [gnutls-dev] [PATCH] Fixing OpenPGP keyring import (again)
Date: Tue, 22 May 2007 09:46:53 +0200
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux)


Timo Schulz <address@hidden> writes:

> Actually the function code also contains another error.
> Due to the nature of cdk_keydb_new_from_stream(),
> the function do not close the stream at the end, cdk_keydb_close(),
> itself, so we need to store it separately and close it
> later.

Hmm, good point!

There's still something wrong with what you checked in, though:

  err = cdk_stream_tmp_from_mem (data->data, data->size, &input);
  if (!err)
    err = cdk_stream_set_armor_flag (input, 0);
  if (!err)
    err = cdk_keydb_new_from_stream (&keyring->db, 0, input);  
  if (err)
      cdk_stream_close (input);
      gnutls_assert ();

That won't work if `cdk_stream_tmp_from_mem ()' returns an error.

How about ChangeLog entries BTW?

> I already modified the code but I still need to test it.

Please, run the tests in Guile-GnuTLS 0.1 [*].  You need Guile 1.8 and
then run "make check".

I'm currently unable to compile GnuTLS from HEAD because of something
fishy going on with Gnulib:

  $ make CFLAGS='-g -Wall' -j2
  make: *** No rule to make target `gl/m4/getpass.m4', needed by `'. 

  $ find -name getpass.m4

  $ grep -r getpass gl/
  gl/Makefile:# Reproduce by: gnulib-tool --import --dir=. 
--local-dir=gl/override --lib=libgnu --source-base=gl --m4-base=gl/m4 
--doc-base=doc --aux-dir=. --avoid=snprintf --avoid=vasnprintf --libtool --macro-prefix=gl arpa_inet fdl gendocs 
getaddrinfo getline getpass gpl inet_ntop inet_pton lgpl maintainer-makefile 
  gl/Makefile:    $(top_srcdir)/gl/m4/getpass.m4 \
  gl/Makefile:    getdelim.h getline.c getline.h getpass.c getpass.h 
inet_ntop.c \
  gl/Makefile:    getline.c getpass.c inet_ntop.c inet_pton.c readline.c \
  gl/Makefile:include ./$(DEPDIR)/getpass.Plo
  gl/ $(top_srcdir)/lgl/m4/getpass.m4 \

So, for some reason, the generated `Makefile' gets the path to
`getpass.m4' wrong, although `' has the right one.

Running `autoreconf', `automake' or `gnulib-tool --update' doesn't
help.  Any idea?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]