fix(module): fix potential memory leak, when protocol error / ngx again

joelwurtz · joelwurtz · commit ca2e27b1e74a · 2026-04-03T13:54:09.000+02:00
diff --git a/src/redirectionio_protocol.c b/src/redirectionio_protocol.c
@@ -97,6 +97,8 @@ apr_status_t redirectionio_protocol_match(redirectionio_connection *conn, redire
     rv = redirectionio_send_protocol_header(conn, project_key, REDIRECTIONIO_PROTOCOL_COMMAND_MATCH_ACTION, r);
 
     if (rv != APR_SUCCESS) {
+        free((void *)request_serialized);
+
         return rv;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -97,6 +97,8 @@ apr_status_t redirectionio_protocol_match(redirectionio_connection *conn, redire`
`97`	`97`	`rv = redirectionio_send_protocol_header(conn, project_key, REDIRECTIONIO_PROTOCOL_COMMAND_MATCH_ACTION, r);`
`98`	`98`
`99`	`99`	`if (rv != APR_SUCCESS) {`
	`100`	`+ free((void *)request_serialized);`
	`101`	`+`
`100`	`102`	`return rv;`
`101`	`103`	`}`
`102`	`104`