Skip to content

Commit 5746823

Browse files
committed
Stabilize modules, fix memory leaks, add chaos proxy
1 parent 96ec7ef commit 5746823

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/mod_redirectionio.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,14 +189,14 @@ static apr_status_t redirectionio_filter_header_filtering(ap_filter_t *f, apr_bu
189189
redirectionio_connection* conn = redirectionio_acquire_connection(config, f->r->pool);
190190

191191
if (conn == NULL) {
192-
return DECLINED;
192+
return ap_pass_brigade(f->next, bb);;
193193
}
194194

195195
// Send headers
196196
if (redirectionio_protocol_send_filter_headers(conn, ctx, f->r, config->project_key) != APR_SUCCESS) {
197197
redirectionio_invalidate_connection(conn, config, f->r->pool);
198198

199-
return DECLINED;
199+
return ap_pass_brigade(f->next, bb);;
200200
}
201201

202202
redirectionio_release_connection(conn, config, f->r->pool);

src/redirectionio_protocol.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,7 @@ apr_status_t redirectionio_protocol_send_filter_headers(redirectionio_connection
193193
}
194194

195195
dst = cJSON_PrintUnformatted(query);
196+
cJSON_Delete(query);
196197

197198
clen = sizeof(COMMAND_FILTER_HEADER_NAME);
198199
rv = apr_socket_send(conn->rio_sock, COMMAND_FILTER_HEADER_NAME, &clen);
@@ -205,6 +206,7 @@ apr_status_t redirectionio_protocol_send_filter_headers(redirectionio_connection
205206

206207
wlen = strlen(dst) + 1;
207208
rv = apr_socket_send(conn->rio_sock, dst, &wlen);
209+
free((void *)dst);
208210

209211
if (rv != APR_SUCCESS) {
210212
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, "mod_redirectionio: Error sending filter headers command data: %s", apr_strerror(rv, errbuf, sizeof(errbuf)));

0 commit comments

Comments
 (0)