line 232:s_env[i+1] = tcn_get_string(e, str); this will crash the java vm for a violate access error.
What are you trying to do?
He tries to launch an external program with a Proc.create() call, I think. I see two problems in the current implementation of Proc::create in proc.c of native 1.1: 1) if (env) es = (*e)->GetArrayLength(e, args); calculates the length of the "args" array, where "env" array is expected. I think it is the actual cause of access violation error. 2) s_env[i+1] = tcn_get_string(e, str); as OP noted, should be s_env[i] = tcn_get_string(e, str); Note, that if the "[i+1]" in the code were correct, there is s_env[i++] = apr_psprintf(p, TCN_PARENT_IDE "=%d", getpid()); assignment later, that will overwrite the last env. variable value in that case. This is probably not an error: I see a cycle of free() calls to clear calls for s_env values. I do not see a free() call for the value assigned with s_env[i++] = apr_psprintf(p, TCN_PARENT_IDE "=%d", getpid()); Probably that is not needed, because it was allocated from a pool?
Created attachment 26464 [details] Patch for proc.c of native/1.1.x. (not tested)
*** Bug 49556 has been marked as a duplicate of this bug. ***
the free() are needed as the tcn_get_string makes malloc().
Fixed by r1056218