Attachment 704 Details for Bug 1166 – patch to fix 1166 according to comment 11 solution B: commit-842ead6-Fix-B.patch

[patch] patch to fix 1166 according to comment 11 solution B: commit-842ead6-Fix-B.patch

commit-842ead6-Fix-B.patch (text/plain), 1.29 KB, created by Xerxes Rånby on 2015-07-08 15:43:10 CEST

(hide)

Description:

Filename:

MIME Type:

Creator: Xerxes Rånby

Created: 2015-07-08 15:43:10 CEST

Size: 1.29 KB

patch

obsolete

>commit 842ead6563f8bc04f562ed32fa56834045f5377d
>Author: Xerxes RÃ¥nby <xerxes@gudinna.com>
>Date:   Wed Jul 8 15:33:56 2015 +0200
>
>    Bug 1166: Fix JavaEmitter: JVMUtil_NewDirectByteBufferCopy
>    
>    Pass jlong capacity as jint to newDirectByteBuffer
>    to prevent typecast truncation of capacity information
>    while crossing jni -> java.
>    
>    Signed-off-by: Xerxes RÃ¥nby <xerxes@gudinna.com>
>
>diff --git a/src/java/com/jogamp/gluegen/JavaEmitter.java b/src/java/com/jogamp/gluegen/JavaEmitter.java
>index 60cd3f4..c434824 100644
>--- a/src/java/com/jogamp/gluegen/JavaEmitter.java
>+++ b/src/java/com/jogamp/gluegen/JavaEmitter.java
>@@ -2558,7 +2558,8 @@ public class JavaEmitter implements GlueEmitter {
>          "        (*env)->FatalError(env, \"initializeImpl() not called\");\n"+
>          "        return NULL;\n"+
>          "    }\n"+
>-         "    jbyteBuffer  = (*env)->CallStaticObjectMethod(env, clazzBuffers, cstrBuffersNew, capacity);\n"+
>+         "    jint intcapacity = capacity;\n" +
>+         "    jbyteBuffer  = (*env)->CallStaticObjectMethod(env, clazzBuffers, cstrBuffersNew, intcapacity);\n"+
>          "    byteBufferPtr = (*env)->GetDirectBufferAddress(env, jbyteBuffer);\n"+
>          "    memcpy(byteBufferPtr, source_address, capacity);\n"+
>          "    return jbyteBuffer;\n"+

Actions: View | Diff

Attachments on bug 1166: 699 | 700 | 702 | 704 | 705