patchWeeChat - Patches: patch #7459, Fix a problem with certificate...

 
 

patch #7459: Fix a problem with certificate verification.

Submitted by:  None
Submitted on:  Wed 09 Feb 2011 05:39:41 PM UTC  
 
Category: otherPriority: 7 - High
Status: DonePrivacy: Public
Assigned to: Sebastien Helleu <flashcode>Originator Email: -unavailable-
Open/Closed: ClosedIRC nick: Gu1

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 14 Feb 2011 11:21:27 PM UTC, comment #2:

Here is another version of the patch that doesn't modify the API. It just adds another parameter to the gnutls_callback_t prototype that specify the mode (either "verify" or "set certificate").

(file #22689)

Anonymous
Wed 09 Feb 2011 05:46:07 PM UTC, comment #1:

oh, and i forgot the POC:
$ openssl genrsa -out server.key 4096
$ openssl req -new -key server.key -out server.csr
$ openssl x509 -req -days 365 -in server.csr -signkey server.key -out server.crt
$ openssl dhparam -outform PEM -out dhparam.pem 4096
$ openssl s_server -cert server.crt -key server.key -dhparam dhparam.pem -accept 6697

then try to connect on 127.0.0.1/6697 -ssl and there will be no verification of the server certificate.
If we configure openssl to request a client certificate using s_server's '-verify' option, then weechat will check the certificate (and detect it as self-signed).

$ openssl s_server -cert server.crt -key server.key -dhparam dhparam.pem -accept 6697 -verify yes

Anonymous
Wed 09 Feb 2011 05:39:41 PM UTC, original submission:

It might be possible to fix the problem without changing the API by calling the gnutls_cb (=irc_server_gnutls_callback) function twice, first to verify the server's certificate and then to set the client certificate ?

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by flashcode (Updated the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 05 Apr 2012 12:10:43 PM UTCflashcodeOpen/ClosedOpen=>Closed
    Wed 02 Mar 2011 02:28:01 PM UTCflashcodeCategoryNone=>other
      StatusNone=>Done
      Assigned toNone=>flashcode
    Mon 14 Feb 2011 11:21:27 PM UTCNoneAttached File-=>Added 0001-Fix-a-problem-with-certificate-verification-v2.patch, #22689
    Wed 09 Feb 2011 05:39:41 PM UTCNoneAttached File-=>Added 0001-Fix-a-problem-with-certificate-verification.patch, #22636

    Back to the top


    Powered by Savane 3.1-cleanup1