Superficial changes to make the exclude/include code easier to understand
This commit is contained in:
parent
f86bbfaddd
commit
0e766183ee
@ -513,25 +513,30 @@ void append_file_to_list(char *filename, char ***name_list, int *num_of_entries)
|
|||||||
if (line_ptr) {
|
if (line_ptr) {
|
||||||
*line_ptr = '\0';
|
*line_ptr = '\0';
|
||||||
}
|
}
|
||||||
*name_list = realloc(*name_list, sizeof(char *) * (*num_of_entries + 1));
|
*name_list = realloc(*name_list, sizeof(char *) * (*num_of_entries + 2));
|
||||||
(*name_list)[*num_of_entries] = xstrdup(line);
|
(*name_list)[*num_of_entries] = xstrdup(line);
|
||||||
(*num_of_entries)++;
|
(*num_of_entries)++;
|
||||||
free(line);
|
free(line);
|
||||||
}
|
}
|
||||||
fclose(src_stream);
|
fclose(src_stream);
|
||||||
|
(*name_list)[*num_of_entries] = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
char **merge_list(char **include_list, char **exclude_list)
|
/*
|
||||||
|
* Create a list of names that are in the include list AND NOT in the exclude lists
|
||||||
|
*/
|
||||||
|
char **list_and_not_list(char **include_list, char **exclude_list)
|
||||||
{
|
{
|
||||||
char **new_include_list = NULL;
|
char **new_include_list = NULL;
|
||||||
int new_include_count = 0;
|
int new_include_count = 0;
|
||||||
int include_count = 0;
|
int include_count = 0;
|
||||||
int exclude_count;
|
int exclude_count;
|
||||||
|
|
||||||
if (include_list == NULL)
|
if (include_list == NULL) {
|
||||||
return exclude_list;
|
return(NULL);
|
||||||
|
}
|
||||||
|
|
||||||
while (include_list[include_count] != NULL) {
|
while (include_list[include_count] != NULL) {
|
||||||
int found = FALSE;
|
int found = FALSE;
|
||||||
exclude_count = 0;
|
exclude_count = 0;
|
||||||
@ -547,12 +552,12 @@ char **merge_list(char **include_list, char **exclude_list)
|
|||||||
new_include_list = realloc(new_include_list, sizeof(char *) * (include_count + 2));
|
new_include_list = realloc(new_include_list, sizeof(char *) * (include_count + 2));
|
||||||
new_include_list[new_include_count] = include_list[include_count];
|
new_include_list[new_include_count] = include_list[include_count];
|
||||||
new_include_count++;
|
new_include_count++;
|
||||||
new_include_list[new_include_count] = NULL;
|
|
||||||
} else {
|
} else {
|
||||||
free(include_list[include_count]);
|
free(include_list[include_count]);
|
||||||
}
|
}
|
||||||
include_count++;
|
include_count++;
|
||||||
}
|
}
|
||||||
|
new_include_list[new_include_count] = NULL;
|
||||||
return(new_include_list);
|
return(new_include_list);
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
@ -572,12 +577,13 @@ int tar_main(int argc, char **argv)
|
|||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
char **exclude_list = NULL;
|
char **exclude_list = NULL;
|
||||||
int exclude_count = 0;
|
int exclude_list_count = 0;
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
FILE *src_stream = NULL;
|
FILE *src_stream = NULL;
|
||||||
FILE *uncompressed_stream = NULL;
|
FILE *uncompressed_stream = NULL;
|
||||||
char **include_list = NULL;
|
char **include_list = NULL;
|
||||||
|
char **the_real_list = NULL;
|
||||||
char *src_filename = NULL;
|
char *src_filename = NULL;
|
||||||
char *dst_prefix = NULL;
|
char *dst_prefix = NULL;
|
||||||
char *file_list_name = NULL;
|
char *file_list_name = NULL;
|
||||||
@ -585,7 +591,7 @@ int tar_main(int argc, char **argv)
|
|||||||
unsigned short untar_funct = 0;
|
unsigned short untar_funct = 0;
|
||||||
unsigned short untar_funct_required = 0;
|
unsigned short untar_funct_required = 0;
|
||||||
unsigned short extract_function = 0;
|
unsigned short extract_function = 0;
|
||||||
int include_count = 0;
|
int include_list_count = 0;
|
||||||
int gunzip_pid;
|
int gunzip_pid;
|
||||||
int gz_fd = 0;
|
int gz_fd = 0;
|
||||||
|
|
||||||
@ -621,13 +627,13 @@ int tar_main(int argc, char **argv)
|
|||||||
/* Exclude or Include files listed in <filename>*/
|
/* Exclude or Include files listed in <filename>*/
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
case 'X':
|
case 'X':
|
||||||
append_file_to_list(optarg, &exclude_list, &exclude_count);
|
append_file_to_list(optarg, &exclude_list, &exclude_list_count);
|
||||||
exclude_list[exclude_count] = NULL;
|
exclude_list[exclude_list_count] = NULL;
|
||||||
break;
|
break;
|
||||||
#endif
|
#endif
|
||||||
case 'T':
|
case 'T':
|
||||||
// by default a list is an include list
|
// by default a list is an include list
|
||||||
append_file_to_list(optarg, &include_list, &include_count);
|
append_file_to_list(optarg, &include_list, &include_list_count);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'C': // Change to dir <optarg>
|
case 'C': // Change to dir <optarg>
|
||||||
@ -677,23 +683,29 @@ int tar_main(int argc, char **argv)
|
|||||||
|
|
||||||
/* Setup an array of filenames to work with */
|
/* Setup an array of filenames to work with */
|
||||||
while (optind < argc) {
|
while (optind < argc) {
|
||||||
include_list = realloc(include_list, sizeof(char *) * (include_count + 2));
|
include_list = realloc(include_list, sizeof(char *) * (include_list_count + 2));
|
||||||
include_list[include_count] = xstrdup(argv[optind]);
|
include_list[include_list_count] = xstrdup(argv[optind]);
|
||||||
include_count++;
|
include_list_count++;
|
||||||
optind++;
|
optind++;
|
||||||
include_list[include_count] = NULL;
|
include_list[include_list_count] = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* By default the include list is the list we act on */
|
||||||
|
the_real_list = include_list;
|
||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
/* Remove excluded files from the include list */
|
|
||||||
if (exclude_list != NULL) {
|
if (exclude_list != NULL) {
|
||||||
/* If both an exclude and include file list were present then
|
|
||||||
* it's an exclude from include list only, if not it's really an
|
|
||||||
* exclude list (and a poor choice of variable names) */
|
|
||||||
if (include_list == NULL) {
|
if (include_list == NULL) {
|
||||||
|
/* the_real_list is an exclude list */
|
||||||
extract_function |= extract_exclude_list;
|
extract_function |= extract_exclude_list;
|
||||||
|
the_real_list = exclude_list;
|
||||||
|
} else {
|
||||||
|
/* Both an exclude and include file list was present,
|
||||||
|
* its an exclude from include list only.
|
||||||
|
* Remove excluded files from the include list
|
||||||
|
*/
|
||||||
|
the_real_list = list_and_not_list(include_list, exclude_list);
|
||||||
}
|
}
|
||||||
include_list = merge_list(include_list, exclude_list);
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
@ -717,7 +729,7 @@ int tar_main(int argc, char **argv)
|
|||||||
uncompressed_stream = src_stream;
|
uncompressed_stream = src_stream;
|
||||||
|
|
||||||
/* extract or list archive */
|
/* extract or list archive */
|
||||||
unarchive(uncompressed_stream, stdout, &get_header_tar, extract_function, dst_prefix, include_list);
|
unarchive(uncompressed_stream, stdout, &get_header_tar, extract_function, dst_prefix, the_real_list);
|
||||||
fclose(uncompressed_stream);
|
fclose(uncompressed_stream);
|
||||||
}
|
}
|
||||||
#ifdef BB_FEATURE_TAR_CREATE
|
#ifdef BB_FEATURE_TAR_CREATE
|
||||||
@ -733,7 +745,7 @@ int tar_main(int argc, char **argv)
|
|||||||
if (extract_function & extract_verbose_list) {
|
if (extract_function & extract_verbose_list) {
|
||||||
verboseFlag = TRUE;
|
verboseFlag = TRUE;
|
||||||
}
|
}
|
||||||
writeTarFile(src_filename, verboseFlag, &argv[argc - 1], include_list);
|
writeTarFile(src_filename, verboseFlag, &argv[argc - 1], the_real_list);
|
||||||
}
|
}
|
||||||
#endif // BB_FEATURE_TAR_CREATE
|
#endif // BB_FEATURE_TAR_CREATE
|
||||||
|
|
||||||
|
56
tar.c
56
tar.c
@ -513,25 +513,30 @@ void append_file_to_list(char *filename, char ***name_list, int *num_of_entries)
|
|||||||
if (line_ptr) {
|
if (line_ptr) {
|
||||||
*line_ptr = '\0';
|
*line_ptr = '\0';
|
||||||
}
|
}
|
||||||
*name_list = realloc(*name_list, sizeof(char *) * (*num_of_entries + 1));
|
*name_list = realloc(*name_list, sizeof(char *) * (*num_of_entries + 2));
|
||||||
(*name_list)[*num_of_entries] = xstrdup(line);
|
(*name_list)[*num_of_entries] = xstrdup(line);
|
||||||
(*num_of_entries)++;
|
(*num_of_entries)++;
|
||||||
free(line);
|
free(line);
|
||||||
}
|
}
|
||||||
fclose(src_stream);
|
fclose(src_stream);
|
||||||
|
(*name_list)[*num_of_entries] = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
char **merge_list(char **include_list, char **exclude_list)
|
/*
|
||||||
|
* Create a list of names that are in the include list AND NOT in the exclude lists
|
||||||
|
*/
|
||||||
|
char **list_and_not_list(char **include_list, char **exclude_list)
|
||||||
{
|
{
|
||||||
char **new_include_list = NULL;
|
char **new_include_list = NULL;
|
||||||
int new_include_count = 0;
|
int new_include_count = 0;
|
||||||
int include_count = 0;
|
int include_count = 0;
|
||||||
int exclude_count;
|
int exclude_count;
|
||||||
|
|
||||||
if (include_list == NULL)
|
if (include_list == NULL) {
|
||||||
return exclude_list;
|
return(NULL);
|
||||||
|
}
|
||||||
|
|
||||||
while (include_list[include_count] != NULL) {
|
while (include_list[include_count] != NULL) {
|
||||||
int found = FALSE;
|
int found = FALSE;
|
||||||
exclude_count = 0;
|
exclude_count = 0;
|
||||||
@ -547,12 +552,12 @@ char **merge_list(char **include_list, char **exclude_list)
|
|||||||
new_include_list = realloc(new_include_list, sizeof(char *) * (include_count + 2));
|
new_include_list = realloc(new_include_list, sizeof(char *) * (include_count + 2));
|
||||||
new_include_list[new_include_count] = include_list[include_count];
|
new_include_list[new_include_count] = include_list[include_count];
|
||||||
new_include_count++;
|
new_include_count++;
|
||||||
new_include_list[new_include_count] = NULL;
|
|
||||||
} else {
|
} else {
|
||||||
free(include_list[include_count]);
|
free(include_list[include_count]);
|
||||||
}
|
}
|
||||||
include_count++;
|
include_count++;
|
||||||
}
|
}
|
||||||
|
new_include_list[new_include_count] = NULL;
|
||||||
return(new_include_list);
|
return(new_include_list);
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
@ -572,12 +577,13 @@ int tar_main(int argc, char **argv)
|
|||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
char **exclude_list = NULL;
|
char **exclude_list = NULL;
|
||||||
int exclude_count = 0;
|
int exclude_list_count = 0;
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
FILE *src_stream = NULL;
|
FILE *src_stream = NULL;
|
||||||
FILE *uncompressed_stream = NULL;
|
FILE *uncompressed_stream = NULL;
|
||||||
char **include_list = NULL;
|
char **include_list = NULL;
|
||||||
|
char **the_real_list = NULL;
|
||||||
char *src_filename = NULL;
|
char *src_filename = NULL;
|
||||||
char *dst_prefix = NULL;
|
char *dst_prefix = NULL;
|
||||||
char *file_list_name = NULL;
|
char *file_list_name = NULL;
|
||||||
@ -585,7 +591,7 @@ int tar_main(int argc, char **argv)
|
|||||||
unsigned short untar_funct = 0;
|
unsigned short untar_funct = 0;
|
||||||
unsigned short untar_funct_required = 0;
|
unsigned short untar_funct_required = 0;
|
||||||
unsigned short extract_function = 0;
|
unsigned short extract_function = 0;
|
||||||
int include_count = 0;
|
int include_list_count = 0;
|
||||||
int gunzip_pid;
|
int gunzip_pid;
|
||||||
int gz_fd = 0;
|
int gz_fd = 0;
|
||||||
|
|
||||||
@ -621,13 +627,13 @@ int tar_main(int argc, char **argv)
|
|||||||
/* Exclude or Include files listed in <filename>*/
|
/* Exclude or Include files listed in <filename>*/
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
case 'X':
|
case 'X':
|
||||||
append_file_to_list(optarg, &exclude_list, &exclude_count);
|
append_file_to_list(optarg, &exclude_list, &exclude_list_count);
|
||||||
exclude_list[exclude_count] = NULL;
|
exclude_list[exclude_list_count] = NULL;
|
||||||
break;
|
break;
|
||||||
#endif
|
#endif
|
||||||
case 'T':
|
case 'T':
|
||||||
// by default a list is an include list
|
// by default a list is an include list
|
||||||
append_file_to_list(optarg, &include_list, &include_count);
|
append_file_to_list(optarg, &include_list, &include_list_count);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'C': // Change to dir <optarg>
|
case 'C': // Change to dir <optarg>
|
||||||
@ -677,23 +683,29 @@ int tar_main(int argc, char **argv)
|
|||||||
|
|
||||||
/* Setup an array of filenames to work with */
|
/* Setup an array of filenames to work with */
|
||||||
while (optind < argc) {
|
while (optind < argc) {
|
||||||
include_list = realloc(include_list, sizeof(char *) * (include_count + 2));
|
include_list = realloc(include_list, sizeof(char *) * (include_list_count + 2));
|
||||||
include_list[include_count] = xstrdup(argv[optind]);
|
include_list[include_list_count] = xstrdup(argv[optind]);
|
||||||
include_count++;
|
include_list_count++;
|
||||||
optind++;
|
optind++;
|
||||||
include_list[include_count] = NULL;
|
include_list[include_list_count] = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* By default the include list is the list we act on */
|
||||||
|
the_real_list = include_list;
|
||||||
|
|
||||||
#ifdef BB_FEATURE_TAR_EXCLUDE
|
#ifdef BB_FEATURE_TAR_EXCLUDE
|
||||||
/* Remove excluded files from the include list */
|
|
||||||
if (exclude_list != NULL) {
|
if (exclude_list != NULL) {
|
||||||
/* If both an exclude and include file list were present then
|
|
||||||
* it's an exclude from include list only, if not it's really an
|
|
||||||
* exclude list (and a poor choice of variable names) */
|
|
||||||
if (include_list == NULL) {
|
if (include_list == NULL) {
|
||||||
|
/* the_real_list is an exclude list */
|
||||||
extract_function |= extract_exclude_list;
|
extract_function |= extract_exclude_list;
|
||||||
|
the_real_list = exclude_list;
|
||||||
|
} else {
|
||||||
|
/* Both an exclude and include file list was present,
|
||||||
|
* its an exclude from include list only.
|
||||||
|
* Remove excluded files from the include list
|
||||||
|
*/
|
||||||
|
the_real_list = list_and_not_list(include_list, exclude_list);
|
||||||
}
|
}
|
||||||
include_list = merge_list(include_list, exclude_list);
|
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
@ -717,7 +729,7 @@ int tar_main(int argc, char **argv)
|
|||||||
uncompressed_stream = src_stream;
|
uncompressed_stream = src_stream;
|
||||||
|
|
||||||
/* extract or list archive */
|
/* extract or list archive */
|
||||||
unarchive(uncompressed_stream, stdout, &get_header_tar, extract_function, dst_prefix, include_list);
|
unarchive(uncompressed_stream, stdout, &get_header_tar, extract_function, dst_prefix, the_real_list);
|
||||||
fclose(uncompressed_stream);
|
fclose(uncompressed_stream);
|
||||||
}
|
}
|
||||||
#ifdef BB_FEATURE_TAR_CREATE
|
#ifdef BB_FEATURE_TAR_CREATE
|
||||||
@ -733,7 +745,7 @@ int tar_main(int argc, char **argv)
|
|||||||
if (extract_function & extract_verbose_list) {
|
if (extract_function & extract_verbose_list) {
|
||||||
verboseFlag = TRUE;
|
verboseFlag = TRUE;
|
||||||
}
|
}
|
||||||
writeTarFile(src_filename, verboseFlag, &argv[argc - 1], include_list);
|
writeTarFile(src_filename, verboseFlag, &argv[argc - 1], the_real_list);
|
||||||
}
|
}
|
||||||
#endif // BB_FEATURE_TAR_CREATE
|
#endif // BB_FEATURE_TAR_CREATE
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user