Fix memory leak

Author: joelwurtz

src/mod_redirectionio.c

@@ -276,11 +276,11 @@ static apr_status_t redirectionio_filter_body_filtering(ap_filter_t *f, apr_buck
             return ap_pass_brigade(f->next, bb);
         }
 
-        input.data = malloc(input.len);
-        memcpy(input.data, input_bucket, input.len);
-
         // Send bucket
         if (input.len > 0) {
+            input.data = malloc(input.len);
+            memcpy(input.data, input_bucket, input.len);
+
             output = redirectionio_action_body_filter_filter(ctx->body_filter, input);
 
             // Create a new one

src/redirectionio_protocol.c

@@ -159,6 +159,8 @@ apr_status_t redirectionio_protocol_log(redirectionio_connection *conn, redirect
     rv = redirectionio_send_protocol_header(conn, project_key, REDIRECTIONIO_PROTOCOL_COMMAND_LOG, r);
 
     if (rv != APR_SUCCESS) {
+        free((char *)log);
+
         return rv;
     }
 
@@ -167,6 +169,7 @@ apr_status_t redirectionio_protocol_log(redirectionio_connection *conn, redirect
 
     if (rv != APR_SUCCESS) {
         ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, "mod_redirectionio: Error sending log command length: %s", apr_strerror(rv, errbuf, sizeof(errbuf)));
+        free((char *)log);
 
         return rv;
     }
@@ -175,10 +178,13 @@ apr_status_t redirectionio_protocol_log(redirectionio_connection *conn, redirect
 
     if (rv != APR_SUCCESS) {
         ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, "mod_redirectionio: Error sending log command data: %s", apr_strerror(rv, errbuf, sizeof(errbuf)));
+        free((char *)log);
 
         return rv;
     }
 
+    free((char *)log);
+
     return APR_SUCCESS;
 }