Net fixes

base/sources/backends/android_net.c

@@ -30,7 +30,7 @@ void iron_https_request(const char *url_base, const char *url_path, const char *
     jbyte *elements = (*env)->GetByteArrayElements(env, bytes_array, NULL);
     if (elements != NULL) {
         callback((char *)elements, callbackdata);
-        // (*env)->ReleaseByteArrayElements(env, bytes_array, elements, JNI_ABORT);
+        (*env)->ReleaseByteArrayElements(env, bytes_array, elements, JNI_ABORT);
     }
 
     (*vm)->DetachCurrentThread(vm);

base/sources/backends/apple_net.m

@@ -19,11 +19,11 @@ void iron_https_request(const char *url_base, const char *url_path, const char *
 	urlstring = [urlstring stringByAppendingString:[NSString stringWithUTF8String:url_path]];
 	NSURL *aUrl = [NSURL URLWithString:urlstring];
 	NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:aUrl];
+	request.HTTPMethod = method == IRON_HTTP_GET ? @"GET" : @"POST";
 	if (data != 0) {
 		NSString *datastring = [NSString stringWithUTF8String:data];
 		request.HTTPBody = [datastring dataUsingEncoding:NSUTF8StringEncoding];
 	}
-	request.HTTPMethod = method == IRON_HTTP_GET ? @"GET" : @"POST";
 
 	NSURLSessionDataTask *dataTask = [
 		session dataTaskWithRequest:request

base/sources/backends/posix_net.c

@@ -59,6 +59,7 @@ void iron_https_request(const char *url_base, const char *url_path, const char *
 			buf = realloc(buf, buf_len);
 		}
 	}
+	buf[pos] = '\0';
 
 	// Parse
 	const char *body_ptr = "";

base/sources/backends/windows_net.c

@@ -25,20 +25,19 @@ void iron_https_request(const char *url_base, const char *url_path, const char *
 	if (hConnect) {
 		wchar_t wurl_path[4096];
 		MultiByteToWideChar(CP_UTF8, 0, url_path, -1, wurl_path, 4096);
-		hRequest =
-		    WinHttpOpenRequest(hConnect, method == IRON_HTTP_GET ? L"GET" : L"POST", wurl_path, NULL, WINHTTP_NO_REFERER, WINHTTP_DEFAULT_ACCEPT_TYPES, WINHTTP_FLAG_SECURE);
+		hRequest = WinHttpOpenRequest(hConnect, method == IRON_HTTP_GET ? L"GET" : L"POST", wurl_path, NULL, WINHTTP_NO_REFERER, WINHTTP_DEFAULT_ACCEPT_TYPES, WINHTTP_FLAG_SECURE);
 	}
 
 	BOOL bResults = FALSE;
-
 	if (hRequest) {
 		DWORD optionalLength = (data != 0 && strlen(data) > 0) ? (DWORD)strlen(data) : 0;
 		bResults = WinHttpSendRequest(hRequest, WINHTTP_NO_ADDITIONAL_HEADERS, 0,
 		                              data == 0 ? WINHTTP_NO_REQUEST_DATA : (LPVOID)data, optionalLength, optionalLength, 0);
 	}
 
-	if (bResults)
+	if (bResults) {
 		bResults = WinHttpReceiveResponse(hRequest, NULL);
+	}
 
 	int returnDataIndex = 0;
 	if (bResults) {
@@ -50,14 +49,8 @@ void iron_https_request(const char *url_base, const char *url_path, const char *
 			}
 
 			if ((int)dwSize + 1 > returnDataSize - returnDataIndex) {
-				int newReturnDataSize = (returnDataIndex + dwSize + 1) * 2;
-				char *newReturnData = (char *)malloc(newReturnDataSize);
-				if (newReturnData == 0) {
-					iron_error("Out of memory\n");
-				}
-				memcpy(newReturnData, returnData, returnDataSize);
-				returnDataSize = newReturnDataSize;
-				returnData = newReturnData;
+				returnDataSize = (returnDataIndex + dwSize + 1) * 2;
+				returnData = realloc(returnData, returnDataSize);
 			}
 
 			DWORD dwDownloaded = 0;

base/sources/iron.h

@@ -1378,7 +1378,8 @@ void _http_callback(const char *body, void *callback_data) {
 	if (body != NULL) {
 		buffer = malloc(sizeof(buffer_t));
 		buffer->length = cbd->size > 0 ? cbd->size : strlen(body);
-		buffer->buffer = body;
+		buffer->buffer = malloc(buffer->length);
+		memcpy(buffer->buffer, body, buffer->length);
 	}
 	cbd->func(cbd->url, buffer);
 	free(cbd);