diff options
author | Kamil Dudka <kdudka@redhat.com> | 2009-11-12 10:54:10 +0000 |
---|---|---|
committer | Kamil Dudka <kdudka@redhat.com> | 2009-11-12 10:54:10 +0000 |
commit | d547d00f2cfa69e07d6873bbebbc82f3132f1b82 (patch) | |
tree | d1ecf84c040135fd2d0793bc81bc40ff694106a7 | |
parent | 668dc0ad87622fe104b5dde5876a8030a2aeb213 (diff) |
- Kevin Baughman provided a fix preventing libcurl-NSS from crash on doubly
closed NSPR descriptor. The issue was hard to find, reported several times
before and always closed unresolved. More info at the RH bug:
https://bugzilla.redhat.com/534176
-rw-r--r-- | CHANGES | 6 | ||||
-rw-r--r-- | lib/nss.c | 2 |
2 files changed, 8 insertions, 0 deletions
@@ -6,6 +6,12 @@ Changelog +Kamil Dudka (12 Nov 2009) +- Kevin Baughman provided a fix preventing libcurl-NSS from crash on doubly + closed NSPR descriptor. The issue was hard to find, reported several times + before and always closed unresolved. More info at the RH bug: + https://bugzilla.redhat.com/534176 + Yang Tse (11 Nov 2009) - Marco Maggi reported that compilation failed when configured --with-gssapi and GNU GSS installed due to a missing mutual exclusion of header files in @@ -1227,7 +1227,9 @@ CURLcode Curl_nss_connect(struct connectdata *conn, int sockindex) connssl->handle = SSL_ImportFD(model, connssl->handle); if(!connssl->handle) goto error; + PR_Close(model); /* We don't need this any more */ + model = NULL; /* This is the password associated with the cert that we're using */ if (data->set.str[STRING_KEY_PASSWD]) { |