]> source.dussan.org Git - rspamd.git/commitdiff
[Fix] Fix memory leak in HTTP maps
authorVsevolod Stakhov <vsevolod@highsecure.ru>
Fri, 27 Jan 2017 18:02:58 +0000 (18:02 +0000)
committerVsevolod Stakhov <vsevolod@highsecure.ru>
Fri, 27 Jan 2017 18:03:33 +0000 (18:03 +0000)
Issue: #1332

src/libutil/map.c
src/libutil/ssl_util.c

index 6fb04d4e5af4ca1b80c575408d45fa4ab67aed69..72939aa7fd492acdbbb84100c82334e77dd31bba 100644 (file)
@@ -97,7 +97,8 @@ write_http_request (struct http_callback_data *cbd)
                }
 
                if (cbd->stage == map_load_file) {
-                       msg->url = rspamd_fstring_new_init (cbd->data->path, strlen (cbd->data->path));
+                       msg->url = rspamd_fstring_append (msg->url,
+                                       cbd->data->path, strlen (cbd->data->path));
 
                        if (cbd->check &&
                                        cbd->data->last_checked != 0 && cbd->stage == map_load_file) {
@@ -107,11 +108,13 @@ write_http_request (struct http_callback_data *cbd)
                        }
                }
                else if (cbd->stage == map_load_pubkey) {
-                       msg->url = rspamd_fstring_new_init (cbd->data->path, strlen (cbd->data->path));
+                       msg->url = rspamd_fstring_append (msg->url,
+                                       cbd->data->path, strlen (cbd->data->path));
                        msg->url = rspamd_fstring_append (msg->url, ".pub", 4);
                }
                else if (cbd->stage == map_load_signature) {
-                       msg->url = rspamd_fstring_new_init (cbd->data->path, strlen (cbd->data->path));
+                       msg->url = rspamd_fstring_append (msg->url,
+                                       cbd->data->path, strlen (cbd->data->path));
                        msg->url = rspamd_fstring_append (msg->url, ".sig", 4);
                }
                else {
index c320dfd29bdde55ea902020238522cf380150ce3..9913e48d3ea6e324c0dd25d70cd23fbf6dce3c94 100644 (file)
@@ -16,6 +16,7 @@
 
 #include "config.h"
 #include "libutil/util.h"
+#include "libutil/logger.h"
 #include "ssl_util.h"
 
 #include <openssl/ssl.h>
@@ -362,7 +363,7 @@ rspamd_ssl_event_handler (gint fd, short what, gpointer ud)
                                c->handler (fd, EV_WRITE, c->handler_data);
                        }
                        else {
-                               g_assert (0);
+                               return;
                        }
                }
                else {
@@ -700,6 +701,7 @@ void
 rspamd_ssl_connection_free (struct rspamd_ssl_connection *conn)
 {
        if (conn) {
+               SSL_shutdown (conn->ssl);
                SSL_free (conn->ssl);
 
                if (conn->hostname) {