gnutls-devel
[Top][All Lists]
Advanced

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

[sr #108123] [PATCH] Fix trust_list_add_system_trust on Windows


From: Marti
Subject: [sr #108123] [PATCH] Fix trust_list_add_system_trust on Windows
Date: Mon, 03 Sep 2012 19:10:59 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20100101 Firefox/15.0

URL:
  <http://savannah.gnu.org/support/?108123>

                 Summary: [PATCH] Fix trust_list_add_system_trust on Windows
                 Project: GnuTLS
            Submitted by: intgr
            Submitted on: Mon 03 Sep 2012 10:10:58 PM EEST
                Category: Core library
                Priority: 5 - Normal
                Severity: 4 - Important
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
        Operating System: Microsoft Windows

    _______________________________________________________

Details:

Apparently gnutls_x509_trust_list_add_system_trust is broken on Windows -- it
never manages to import any certificates on the machines I tried it on (always
returning 0). Upon closer inspection, the gnutls_x509_trust_list_add_trust_mem
function always returned -49 (GNUTLS_E_NO_CERTIFICATE_FOUND). Looking further
in the function, the problem was obvious...

Patch attached:
---
Fix gnutls_x509_trust_list_add_trust_mem with DER-format certificates.

The function took a "type" argument and then happily proceeded to ignore
it and try PEM format anyway.

Most importantly, this makes gnutls_x509_trust_list_add_system_trust
work on Windows, which loads DER certificates using this function.  I'll
be damned if that actually ever worked properly -- certainly not in any
git version. :)

Also added test for gnutls_x509_trust_list_add_trust_mem.
---

Also available on github:
https://github.com/intgr/gnutls/tree/win32-systrust-fix

I agree to assign my copyrights on this patch to the FSF.




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Mon 03 Sep 2012 10:10:58 PM EEST  Name: win32-systrust-fix.patch  Size:
6kB   By: intgr

<http://savannah.gnu.org/support/download.php?file_id=26466>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/support/?108123>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

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