WIP Fixing tests for nginx

joelwurtz · joelwurtz · commit 20ea3804d1b2 · 2020-05-06T14:26:48.000+02:00
diff --git a/src/redirectionio_protocol.c b/src/redirectionio_protocol.c
@@ -106,7 +106,7 @@ apr_status_t redirectionio_protocol_match(redirectionio_connection *conn, redire
     }
 
     // Read action
-    action_serialized = apr_palloc(r->pool, alen);
+    action_serialized = apr_palloc(r->pool, alen + 1);
 
     if (action_serialized == NULL) {
         return APR_EGENERAL;
@@ -330,6 +330,8 @@ static apr_status_t redirectionio_read_string(redirectionio_connection *conn, ch
         srlen = buf_size - sdrlen;
     }
 
+    *(string + buf_size) = '\0';
+
     return APR_SUCCESS;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ apr_status_t redirectionio_protocol_match(redirectionio_connection *conn, redire`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`// Read action`
`109`		`- action_serialized = apr_palloc(r->pool, alen);`
	`109`	`+ action_serialized = apr_palloc(r->pool, alen + 1);`
`110`	`110`
`111`	`111`	`if (action_serialized == NULL) {`
`112`	`112`	`return APR_EGENERAL;`
`@@ -330,6 +330,8 @@ static apr_status_t redirectionio_read_string(redirectionio_connection *conn, ch`
`330`	`330`	`srlen = buf_size - sdrlen;`
`331`	`331`	`}`
`332`	`332`
	`333`	`+ *(string + buf_size) = '\0';`
	`334`	`+`
`333`	`335`	`return APR_SUCCESS;`
`334`	`336`	`}`
`335`	`337`