GNU bug report logs - #18664
Values for gnutls-verify-error produced by customize are not parsed correctly

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 18664 in the body.
You can then email your comments to 18664 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Toke Høiland-Jørgensen <toke <at> toke.dk>
To: bug-gnu-emacs <at> gnu.org
Subject: Values for gnutls-verify-error produced by customize are not parsed
 correctly
Date: Wed, 08 Oct 2014 17:07:21 +0200

Using customize to set gnutls-verify-error, a doubly-nested list of
symbols ends up getting passed to gnutls-boot, which then subsequently
fails all the membership tests on it. This seems to stem from the fact
that customize produces list entries of the form '((".*" (:hostname))) while
gnutls-negotiate passes the cdr of each matched entry to cl-mapcan.

Found this while testing in trunk.

-Toke

Message #8 received at 18664 <at> debbugs.gnu.org (full text, mbox):

From: Ted Zlatanov <tzz <at> lifelogs.com>
To: Toke Høiland-Jørgensen <toke <at> toke.dk>
Cc: 18664 <at> debbugs.gnu.org
Subject: Re: bug#18664: Values for gnutls-verify-error produced by customize
 are not parsed correctly
Date: Fri, 17 Oct 2014 21:03:30 -0400

On Wed, 08 Oct 2014 17:07:21 +0200 Toke Høiland-Jørgensen <toke <at> toke.dk> wrote: 

TH> Using customize to set gnutls-verify-error, a doubly-nested list of
TH> symbols ends up getting passed to gnutls-boot, which then subsequently
TH> fails all the membership tests on it. This seems to stem from the fact
TH> that customize produces list entries of the form '((".*" (:hostname))) while
TH> gnutls-negotiate passes the cdr of each matched entry to cl-mapcan.

Thanks for the report.  I believe it's fixed with the commit below; can
you please test?

Thanks
Ted

------------------------------------------------------------
revno: 118145
committer: Ted Zlatanov <tzz <at> lifelogs.com>
branch nick: quickfixes
timestamp: Fri 2014-10-17 21:02:30 -0400
message:
  net/gnutls.el: pass correct data to `gnutls-boot'.
  
  * net/gnutls.el (gnutls-negotiate): Don't use cl-mapcan; pass
  correct data to `gnutls-boot'.
  Reported by Toke Høiland-Jørgensen <toke <at> toke.dk>.

Message #11 received at 18664 <at> debbugs.gnu.org (full text, mbox):

From: Toke Høiland-Jørgensen <toke <at> toke.dk>
To: 18664 <at> debbugs.gnu.org
Subject: Re: bug#18664: Values for gnutls-verify-error produced by customize
 are not parsed correctly
Date: Thu, 23 Oct 2014 17:03:13 -0700

Ted Zlatanov <tzz <at> lifelogs.com> writes:

> Thanks for the report. I believe it's fixed with the commit below; can
> you please test?

Yep, this seems to fix it. Thanks! :)

-Toke

Message #16 received at 18664-done <at> debbugs.gnu.org (full text, mbox):

From: Ted Zlatanov <tzz <at> lifelogs.com>
To: Toke Høiland-Jørgensen <toke <at> toke.dk>
Cc: 18664-done <at> debbugs.gnu.org
Subject: Re: bug#18664: Values for gnutls-verify-error produced by customize
 are not parsed correctly
Date: Fri, 24 Oct 2014 09:44:45 -0400

On Thu, 23 Oct 2014 17:03:13 -0700 Toke Høiland-Jørgensen <toke <at> toke.dk> wrote: 

TH> Ted Zlatanov <tzz <at> lifelogs.com> writes:
>> Thanks for the report. I believe it's fixed with the commit below; can
>> you please test?

TH> Yep, this seems to fix it. Thanks! :)

All right, marking as done.  Thank you for the help.

(I haven't forgotten about the TOFU patch, sorry for the delay :)

Ted

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.