mirror of
https://github.com/eclipse-cdt/cdt
synced 2025-06-08 10:16:03 +02:00
Fixes from Alex
This commit is contained in:
parent
280c95fcfe
commit
bb4a4631b5
1 changed files with 53 additions and 35 deletions
|
@ -22,7 +22,7 @@
|
||||||
#include "jni.h"
|
#include "jni.h"
|
||||||
#include "io.h"
|
#include "io.h"
|
||||||
|
|
||||||
#define DEBUG_MONITOR
|
// #define DEBUG_MONITOR
|
||||||
|
|
||||||
#define PIPE_SIZE 512
|
#define PIPE_SIZE 512
|
||||||
#define MAX_CMD_SIZE 1024
|
#define MAX_CMD_SIZE 1024
|
||||||
|
@ -156,6 +156,8 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec0
|
||||||
jsize len = (*env) -> GetStringUTFLength(env, item);
|
jsize len = (*env) -> GetStringUTFLength(env, item);
|
||||||
int nCpyLen;
|
int nCpyLen;
|
||||||
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
||||||
|
if(NULL != str)
|
||||||
|
{
|
||||||
if(0 > (nCpyLen = copyTo(szCmdLine + nPos, str, len, MAX_CMD_SIZE - nPos)))
|
if(0 > (nCpyLen = copyTo(szCmdLine + nPos, str, len, MAX_CMD_SIZE - nPos)))
|
||||||
{
|
{
|
||||||
ThrowByName(env, "java/Exception", "Too long command line");
|
ThrowByName(env, "java/Exception", "Too long command line");
|
||||||
|
@ -166,6 +168,7 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec0
|
||||||
++nPos;
|
++nPos;
|
||||||
(*env) -> ReleaseStringUTFChars(env, item, str);
|
(*env) -> ReleaseStringUTFChars(env, item, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
szCmdLine[nPos] = '\0';
|
szCmdLine[nPos] = '\0';
|
||||||
|
|
||||||
|
@ -178,6 +181,8 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec0
|
||||||
jsize len = (*env) -> GetStringUTFLength(env, item);
|
jsize len = (*env) -> GetStringUTFLength(env, item);
|
||||||
int nCpyLen;
|
int nCpyLen;
|
||||||
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
||||||
|
if(NULL != str)
|
||||||
|
{
|
||||||
if(0 > (nCpyLen = copyTo(szEnvBlock + nPos, str, len, MAX_ENV_SIZE - nPos - 1)))
|
if(0 > (nCpyLen = copyTo(szEnvBlock + nPos, str, len, MAX_ENV_SIZE - nPos - 1)))
|
||||||
{
|
{
|
||||||
ThrowByName(env, "java/Exception", "Too many environment variables");
|
ThrowByName(env, "java/Exception", "Too many environment variables");
|
||||||
|
@ -188,6 +193,7 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec0
|
||||||
++nPos;
|
++nPos;
|
||||||
(*env) -> ReleaseStringUTFChars(env, item, str);
|
(*env) -> ReleaseStringUTFChars(env, item, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
szEnvBlock[nPos] = '\0';
|
szEnvBlock[nPos] = '\0';
|
||||||
envBlk = szEnvBlock;
|
envBlk = szEnvBlock;
|
||||||
}
|
}
|
||||||
|
@ -196,10 +202,13 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec0
|
||||||
|
|
||||||
if (dir != 0)
|
if (dir != 0)
|
||||||
{
|
{
|
||||||
const char * str = NULL;
|
const char * str = (*env) -> GetStringUTFChars(env, dir, 0);
|
||||||
cwd = strdup((*env) -> GetStringUTFChars(env, dir, 0));
|
if(NULL != str)
|
||||||
|
{
|
||||||
|
cwd = strdup(str);
|
||||||
(*env) -> ReleaseStringUTFChars(env, dir, str);
|
(*env) -> ReleaseStringUTFChars(env, dir, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
memset(&si, 0, sizeof(si));
|
memset(&si, 0, sizeof(si));
|
||||||
|
@ -355,6 +364,8 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec1
|
||||||
jsize len = (*env) -> GetStringUTFLength(env, item);
|
jsize len = (*env) -> GetStringUTFLength(env, item);
|
||||||
int nCpyLen;
|
int nCpyLen;
|
||||||
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
||||||
|
if(NULL != str)
|
||||||
|
{
|
||||||
if(0 > (nCpyLen = copyTo(szCmdLine + nPos, str, len, MAX_CMD_SIZE - nPos)))
|
if(0 > (nCpyLen = copyTo(szCmdLine + nPos, str, len, MAX_CMD_SIZE - nPos)))
|
||||||
{
|
{
|
||||||
ThrowByName(env, "java/Exception", "Too long command line");
|
ThrowByName(env, "java/Exception", "Too long command line");
|
||||||
|
@ -365,6 +376,7 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec1
|
||||||
++nPos;
|
++nPos;
|
||||||
(*env) -> ReleaseStringUTFChars(env, item, str);
|
(*env) -> ReleaseStringUTFChars(env, item, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
szCmdLine[nPos] = '\0';
|
szCmdLine[nPos] = '\0';
|
||||||
|
|
||||||
|
@ -377,6 +389,8 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec1
|
||||||
jsize len = (*env) -> GetStringUTFLength(env, item);
|
jsize len = (*env) -> GetStringUTFLength(env, item);
|
||||||
int nCpyLen;
|
int nCpyLen;
|
||||||
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
const char * str = (*env) -> GetStringUTFChars(env, item, 0);
|
||||||
|
if(NULL != str)
|
||||||
|
{
|
||||||
if(0 > (nCpyLen = copyTo(szEnvBlock + nPos, str, len, MAX_ENV_SIZE - nPos - 1)))
|
if(0 > (nCpyLen = copyTo(szEnvBlock + nPos, str, len, MAX_ENV_SIZE - nPos - 1)))
|
||||||
{
|
{
|
||||||
ThrowByName(env, "java/Exception", "Too many environment variables");
|
ThrowByName(env, "java/Exception", "Too many environment variables");
|
||||||
|
@ -387,6 +401,7 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec1
|
||||||
++nPos;
|
++nPos;
|
||||||
(*env) -> ReleaseStringUTFChars(env, item, str);
|
(*env) -> ReleaseStringUTFChars(env, item, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
szEnvBlock[nPos] = '\0';
|
szEnvBlock[nPos] = '\0';
|
||||||
envBlk = szEnvBlock;
|
envBlk = szEnvBlock;
|
||||||
}
|
}
|
||||||
|
@ -395,10 +410,13 @@ JNIEXPORT jint JNICALL Java_org_eclipse_cdt_utils_spawner_Spawner_exec1
|
||||||
|
|
||||||
if (dir != 0)
|
if (dir != 0)
|
||||||
{
|
{
|
||||||
const char * str = NULL;
|
const char * str = (*env) -> GetStringUTFChars(env, dir, 0);
|
||||||
cwd = strdup((*env) -> GetStringUTFChars(env, dir, 0));
|
if(NULL != str)
|
||||||
|
{
|
||||||
|
cwd = strdup(str);
|
||||||
(*env) -> ReleaseStringUTFChars(env, dir, str);
|
(*env) -> ReleaseStringUTFChars(env, dir, str);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
memset(&si, 0, sizeof(si));
|
memset(&si, 0, sizeof(si));
|
||||||
|
|
Loading…
Add table
Reference in a new issue