On 12/03/2014 17:47, Mark Sheppard wrote: > Hi Alan, > so L61 should be > if ((bytes = (*env)->NewByteArray(env, i)) == NULL) { > FreeEnvironmentStringsA(blockA); > return NULL; > } That would be good - thanks! -Alan