diff -ur fastjar-0.98.orig/jartool.c fastjar-0.98/jartool.c --- fastjar-0.98.orig/jartool.c 2009-09-06 18:10:47.000000000 -0400 +++ fastjar-0.98/jartool.c 2010-04-28 17:15:09.000000000 -0400 @@ -1730,8 +1730,18 @@ struct stat sbuf; int depth = 0; + if(strncmp((const char *)filename, "/", 1) == 0){ + fprintf(stderr, "Absolute path names are not allowed.\n"); + exit(EXIT_FAILURE); + } + tmp_buff = malloc(sizeof(char) * strlen((const char *)filename)); + if(tmp_buff == NULL) { + fprintf(stderr, "Out of memory.\n"); + exit(EXIT_FAILURE); + } + for(;;){ const ub1 *idx = (const unsigned char *)strchr((const char *)start, '/'); @@ -1749,14 +1759,17 @@ #ifdef DEBUG printf("checking the existance of %s\n", tmp_buff); #endif - if(strcmp(tmp_buff, "..") == 0){ - --depth; - if (depth < 0){ - fprintf(stderr, "Traversal to parent directories during unpacking!\n"); - exit(EXIT_FAILURE); - } - } else if (strcmp(tmp_buff, ".") != 0) - ++depth; + if(strcmp(tmp_buff, "..") == 0 || (strlen(tmp_buff) > 2 && strncmp(tmp_buff + strlen(tmp_buff) - 3, "/..", 3) == 0)){ + --depth; + if (depth < 0){ + fprintf(stderr, "Traversal to parent directories during unpacking!\n"); + exit(EXIT_FAILURE); + } + } else if (strcmp(tmp_buff, ".") == 0 || (strlen(tmp_buff) > 1 && strncmp(tmp_buff + strlen(tmp_buff) - 2, "/.", 2) == 0)){ + /* Do nothing, the current directory is "." */ + } else + ++depth; + if(stat(tmp_buff, &sbuf) < 0){ if(errno != ENOENT) exit_on_error("stat");