Handle LDKStr in other structs and fix string conversion overread

diff --git a/genbindings.py b/genbindings.py
index 4b5ef5dc8bdc96e1de0ff0b85cada71bbb4fe413..3d5bac974c5b563cd50d1d84dfe3d6d2b5199f59 100755 (executable)
--- a/genbindings.py
+++ b/genbindings.py
@@ -260,6 +260,7 @@ def java_c_types(fn_arg, ret_arr_len):
         fn_ty_arg = "Ljava/lang/String;"
         fn_arg = fn_arg[6:].strip()
     elif fn_arg.startswith("LDKStr"):
+        rust_obj = "LDKStr"
         java_ty = "String"
         c_ty = "jstring"
         fn_ty_arg = "Ljava/lang/String;"

diff --git a/java_strings.py b/java_strings.py
index 0c86200e9a1535ec378ad0436374ccadbae62deb..43265f449c2359ebc14d7bcbf803a0f43a0e2d0a 100644 (file)
--- a/java_strings.py
+++ b/java_strings.py
@@ -309,12 +309,12 @@ typedef jbyteArray int8_tArray;
 
 static inline jstring str_ref_to_java(JNIEnv *env, const char* chars, size_t len) {
        // Sadly we need to create a temporary because Java can't accept a char* without a 0-terminator
-       char* err_buf = MALLOC(len + 1, "str conv buf");
-       memcpy(err_buf, chars, len);
-       err_buf[len] = 0;
-       jstring err_conv = (*env)->NewStringUTF(env, chars);
-       FREE(err_buf);
-       return err_conv;
+       char* conv_buf = MALLOC(len + 1, "str conv buf");
+       memcpy(conv_buf, chars, len);
+       conv_buf[len] = 0;
+       jstring ret = (*env)->NewStringUTF(env, conv_buf);
+       FREE(conv_buf);
+       return ret;
 }
 static inline LDKStr java_to_owned_str(JNIEnv *env, jstring str) {
        uint64_t str_len = (*env)->GetStringUTFLength(env, str);