https://issues.apache.org/bugzilla/show_bug.cgi?id=49557
Konstantin Kolinko <knst.koli...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |NEW
--- Comment #2 from Konstantin Kolinko <knst.koli...@gmail.com> 2011-01-06
18:15:41 EST ---
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?
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
