-
Bug
-
Resolution: Fixed
-
P3
-
jfx11, 8
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8240864 | 8u261 | Johan Vos | P3 | Resolved | Fixed | b01 |
JDK-8247978 | jfx11.0.8 | Johan Vos | P3 | Closed | Fixed |
The following call to GetPrimitiveArrayCritical in jpegloader.c is misleading:
jboolean iscopy = FALSE;
jbyte *body = (*env)->GetPrimitiveArrayCritical(env, barray, &iscopy);
It looks like we want to pass "FALSE" to the isCopy parameter of GetPrimitiveArrayCritical, while this field is actually set by the VM when it informs the caller whether a copy is made or not. The jscopy parameter is not used outside this scope.
jboolean iscopy = FALSE;
jbyte *body = (*env)->GetPrimitiveArrayCritical(env, barray, &iscopy);
It looks like we want to pass "FALSE" to the isCopy parameter of GetPrimitiveArrayCritical, while this field is actually set by the VM when it informs the caller whether a copy is made or not. The jscopy parameter is not used outside this scope.
- backported by
-
JDK-8240864 GetPrimitiveArrayCritical passed with hardcoded FALSE value
-
- Resolved
-
-
JDK-8247978 GetPrimitiveArrayCritical passed with hardcoded FALSE value
-
- Closed
-
- blocks
-
JDK-8212034 Potential memory leaks in jpegLoader.c in error case
-
- Resolved
-