From: Gwangho Hwang Date: Sat, 22 Jun 2013 14:26:04 +0000 (+0900) Subject: DEBUG option bug fix and Prevent defect fix X-Git-Tag: submit/tizen_2.2/20130716.103935~1 X-Git-Url: http://review.tizen.org/git/?p=framework%2Fappfw%2Fdebug-launchpad.git;a=commitdiff_plain;h=8d5815a81b305e4ba2ec3d0e39246040ef7905ec DEBUG option bug fix and Prevent defect fix Change-Id: Ie7d665b2fa2ad48bd9ee7c1c2497ea26a16c507d Signed-off-by: Gwangho Hwang --- diff --git a/src/launchpad.c b/src/launchpad.c index 13d3478..22ebe95 100755 --- a/src/launchpad.c +++ b/src/launchpad.c @@ -63,6 +63,7 @@ #define SQLITE_FLUSH_MAX (1048576) /* (1024*1024) */ #define AUL_POLL_CNT 15 #define AUL_PR_NAME 16 +#define APPID_LEN 10 #define PATH_TMP "/tmp" #define PATH_DATA "/data" @@ -137,7 +138,8 @@ _static_ void __set_sdk_env(app_info_from_db* menu_info, char* str) { /* GCOV_PREFIX_STRIP indicates the how many initial directory names */ /* to stripoff the hardwired absolute paths. Default value is 0. */ if (strncmp(str, SDK_CODE_COVERAGE, strlen(str)) == 0) { - strncpy(buf_pkgname,_get_pkgname(menu_info),MAX_LOCAL_BUFSZ); + strncpy(buf_pkgname,_get_pkgname(menu_info),MAX_LOCAL_BUFSZ-1); + buf_pkgname[MAX_LOCAL_BUFSZ-1]='\0'; snprintf(buf, MAX_LOCAL_BUFSZ, PATH_TMP"/%s"PATH_DATA, strtok(buf_pkgname,".")); ret = setenv("GCOV_PREFIX", buf, 1); _D("GCOV_PREFIX : %d", ret); @@ -264,7 +266,10 @@ _static_ char **__add_arg(bundle * kb, char **argv, int *margc, const char *key) if(str_array != NULL) { if(strncmp(key, DLP_K_DEBUG_ARG, strlen(key)) == 0) { argv = (char **) realloc(argv, sizeof(char *) * (*margc+len+2)); - if(!argv) _E("realloc fail"); + if(!argv) { + _E("realloc fail (key = %s)", key); + exit(-1); + } for(i=*margc+len+1; i-(len+1)>=0; i--) { argv[i] = argv[i-(len+1)]; } @@ -275,7 +280,10 @@ _static_ char **__add_arg(bundle * kb, char **argv, int *margc, const char *key) len++; /* gdbserver */ } else { argv = (char **) realloc(argv, sizeof(char *) * (*margc+len+1)); - if(!argv) _E("realloc fail"); + if(!argv) { + _E("realloc fail (key = %s)", key); + exit(-1); + } for(i=0; i