Change read_package_field interface, and rewrite using low level functions
authorGlenn L McGrath <bug1@ihug.co.nz>
Wed, 18 Jul 2001 15:47:21 +0000 (15:47 -0000)
committerGlenn L McGrath <bug1@ihug.co.nz>
Wed, 18 Jul 2001 15:47:21 +0000 (15:47 -0000)
Fixes for a few bugs that have crept into dpkg in the last few days

archival/dpkg.c
archival/dpkg_deb.c
dpkg.c
dpkg_deb.c
include/libbb.h
libbb/libbb.h
libbb/read_package_field.c

index e9afe5f6e66f53328929f083c71b9199cbef7eb9..f207b23ac19356a908357aea6e6d67ac94d1fa12 100644 (file)
@@ -499,34 +499,17 @@ unsigned int fill_package_struct(char *control_buffer)
 {
        common_node_t *new_node = (common_node_t *) xcalloc(1, sizeof(common_node_t));
 
-       char *field;
-       char *field_name;
-       char *field_value;
+       char *field_name = xmalloc(sizeof(char *));
+       char *field_value = xmalloc(sizeof(char *));
        int field_start = 0;
-       int field_length;
-       int seperator_offset;
        int num = -1;
        int buffer_length = strlen(control_buffer);
 
        new_node->version = search_name_hashtable("unknown");
        while (field_start < buffer_length) {
-               field = read_package_field(&control_buffer[field_start]);
-
-               /* Setup start point for next field */
-               field_length = strlen(field);
-               field_start += (field_length + 1);
-
-               seperator_offset = strcspn(field, ":");
-               if (seperator_offset == 0) {
-                       free(field);
-                       continue;
-               }
-               field_name = xstrndup(field, seperator_offset);
-               field_value = field + seperator_offset + 1;
-               field_value += strspn(field_value, " \n\t");
+               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
 
-               /* Should be able to replace this strlen with pointer arithmatic */
-               if (strlen(field_value) == 0) {
+               if (field_name == NULL) {
                        goto fill_package_struct_cleanup; // Oh no, the dreaded goto statement !!
                }
 
@@ -562,7 +545,7 @@ unsigned int fill_package_struct(char *control_buffer)
                }
 fill_package_struct_cleanup:
                free(field_name);
-               free(field);
+               free(field_value);
        }
        if (new_node->version == search_name_hashtable("unknown")) {
                free_package(new_node);
@@ -739,6 +722,23 @@ char *get_depends_field(common_node_t *package, const int depends_type)
        return(depends);
 }
 
+void write_buffer_no_status(FILE *new_status_file, const char *control_buffer)
+{
+       char *name;
+       char *value;
+       int start = 0;
+       while (1) {
+               start += read_package_field(&control_buffer[start], &name, &value);
+               if (name == NULL) {
+                       break;
+               }
+               if (strcmp(name, "Status") != 0) {
+                       fprintf(new_status_file, "%s: %s\n", name, value);
+               }
+       }
+       return;
+}
+
 /* This could do with a cleanup */
 void write_status_file(deb_file_t **deb_file)
 {
@@ -748,11 +748,8 @@ void write_status_file(deb_file_t **deb_file)
        char *status_from_file;
        char *control_buffer = NULL;
        char *tmp_string;
-       char *field;
        int status_num;
-       int field_length;
        int field_start = 0;
-       int buffer_length;
        int write_flag;
        int i = 0;
 
@@ -762,7 +759,6 @@ void write_status_file(deb_file_t **deb_file)
                tmp_string += strspn(tmp_string, " \n\t");
                package_name = xstrndup(tmp_string, strcspn(tmp_string, "\n\0"));
                write_flag = FALSE;
-
                tmp_string = strstr(control_buffer, "Status:");
                if (tmp_string != NULL) {
                        /* Seperate the status value from the control buffer */
@@ -786,20 +782,11 @@ void write_status_file(deb_file_t **deb_file)
                                        i = 0;
                                        while(deb_file[i] != NULL) {
                                                if (strcmp(package_name, name_hashtable[package_hashtable[deb_file[i]->package]->name]) == 0) {
-                                                       char *last_char;
                                                        /* Write a status file entry with a modified status */
                                                        /* remove trailing \n's */
-                                                       while(1) {
-                                                               last_char = last_char_is(deb_file[i]->control_file, '\n');
-                                                               if (last_char) {
-                                                                       *last_char = '\0';
-                                                               } else {
-                                                                       break;
-                                                               }
-                                                       }
-                                                       fputs(deb_file[i]->control_file, new_status_file);
+                                                       write_buffer_no_status(new_status_file, deb_file[i]->control_file);
                                                        set_status(status_num, "ok", 2);
-                                                       fprintf(new_status_file, "\nStatus: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
+                                                       fprintf(new_status_file, "Status: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
                                                        write_flag = TRUE;
                                                        break;
                                                }
@@ -814,16 +801,17 @@ void write_status_file(deb_file_t **deb_file)
                                        /* Only write the Package, Status, Priority and Section lines */
                                        fprintf(new_status_file, "Package: %s\n", package_name);
                                        fprintf(new_status_file, "Status: %s\n", status_from_hashtable);
-                                       buffer_length = strlen(control_buffer);
-                                       while (field_start < buffer_length) {
-                                               field = read_package_field(&control_buffer[field_start]);
-                                               field_length = strlen(field);
-                                               field_start += (field_length + 1);
-                                               if (strncmp(field, "Priority:", 9) == 0) {
-                                                       fprintf(new_status_file, "Priority:%s\n", field + 9);
+
+                                       while (1) {
+                                               char *field_name;
+                                               char *field_value;
+                                               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
+                                               if (field_name == NULL) {
+                                                       break;
                                                }
-                                               if (strncmp(field, "Section:", 8) == 0) {
-                                                       fprintf(new_status_file, "Section:%s\n", field + 8);
+                                               if ((strcmp(field_name, "Priority") == 0) ||
+                                                       (strcmp(field_name, "Section") == 0)) {
+                                                       fprintf(new_status_file, "%s: %s\n", field_name, field_value);
                                                }
                                        }
                                        write_flag = TRUE;
@@ -831,25 +819,26 @@ void write_status_file(deb_file_t **deb_file)
                                }
                                else if (strcmp("config-files", name_hashtable[state_status]) == 0) {
                                        /* only change the status line */
-                                       buffer_length = strlen(control_buffer);
-                                       while (field_start < buffer_length) {
-                                               field = read_package_field(&control_buffer[field_start]);
+//                                     buffer_length = strlen(control_buffer);
+                                       while (1) {
+                                               char *field_name;
+                                               char *field_value;
+                                               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
+                                               if (field_name == NULL) {
+                                                       break;
+                                               }
                                                /* Setup start point for next field */
-                                               field_length = strlen(field);
-                                               field_start += (field_length + 1);
-                                               if (strncmp(field, "Status:", 7) == 0) {
+                                               if (strcmp(field_name, "Status") == 0) {
                                                        fprintf(new_status_file, "Status: %s\n", status_from_hashtable);
                                                } else {
-                                                       fprintf(new_status_file, "%s\n", field);
+                                                       fprintf(new_status_file, "%s: %s\n", field_name, field_value);
                                                }
-                                               free(field);
                                        }
                                        write_flag = TRUE;
                                        fputs("\n", new_status_file);
                                }
                        }
                }
-
                /* If the package from the status file wasnt handle above, do it now*/
                if (write_flag == FALSE) {
                        fprintf(new_status_file, "%s\n\n", control_buffer);
@@ -866,20 +855,9 @@ void write_status_file(deb_file_t **deb_file)
        for(i = 0; deb_file[i] != NULL; i++) {
                status_num = search_status_hashtable(name_hashtable[package_hashtable[deb_file[i]->package]->name]);
                if (strcmp("reinstreq", name_hashtable[get_status(status_num, 2)]) == 0) {
-                       char *last_char;
-                       /* remove trailing \n's */
-                       while(1) {
-                               last_char = last_char_is(deb_file[i]->control_file, '\n');
-                               if (last_char) {
-                                       *last_char = '\0';
-                               } else {
-                                       break;
-                               }
-                       }
-                               
-                       fputs(deb_file[i]->control_file, new_status_file);
+                       write_buffer_no_status(new_status_file, deb_file[i]->control_file);
                        set_status(status_num, "ok", 2);
-                       fprintf(new_status_file, "\nStatus: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
+                       fprintf(new_status_file, "Status: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
                }
        }
        fclose(old_status_file);
@@ -1037,9 +1015,11 @@ char **create_list(const char *filename)
        int length = 0;
        int count = 0;
 
-       list_stream = xfopen(filename, "r");
+       /* dont use [xw]fopen here, handle error ourself */
+       list_stream = fopen(filename, "r");
        if (list_stream == NULL) {
-               return(NULL);
+               *file_list = NULL;
+               return(file_list);
        }
        while (getline(&line, &length, list_stream) != -1) {
                file_list = xrealloc(file_list, sizeof(char *) * (length + 1));
@@ -1053,6 +1033,7 @@ char **create_list(const char *filename)
                length = 0;
        }
        fclose(list_stream);
+
        if (count == 0) {
                return(NULL);
        } else {
@@ -1212,6 +1193,7 @@ void purge_package(const unsigned int package_num)
        if (run_package_script(package_name, "postrm") == -1) {
                error_msg_and_die("postrm fialure.. set status to what?");
        }
+
        /* Change package status */
        set_status(status_num, "purge", 1);
        set_status(status_num, "not-installed", 3);
@@ -1219,7 +1201,6 @@ void purge_package(const unsigned int package_num)
 
 void unpack_package(deb_file_t *deb_file)
 {
-//     const unsigned int package_name_num = package_hashtable[deb_file->package]->name;
        const char *package_name = name_hashtable[package_hashtable[deb_file->package]->name];
        const unsigned int status_num = search_status_hashtable(package_name);
        const unsigned int status_package_num = status_hashtable[status_num]->status;
@@ -1248,7 +1229,6 @@ void unpack_package(deb_file_t *deb_file)
 
        /* Create the list file */
        strcat(info_prefix, "list");
-
        out_stream = xfopen(info_prefix, "w");                  
        deb_extract(deb_file->filename, out_stream, (extract_quiet | extract_data_tar_gz | extract_list), NULL, NULL);
        fclose(out_stream);
@@ -1393,7 +1373,7 @@ extern int dpkg_main(int argc, char **argv)
        /* TODO: check dependencies before removing */
        if ((dpkg_opt & dpkg_opt_force_ignore_depends) != dpkg_opt_force_ignore_depends) {
                if (!check_deps(deb_file, 0, deb_count)) {
-                       error_msg_and_die("Dependency check fialed");
+                       error_msg_and_die("Dependency check failed");
                }
        }
 
@@ -1416,6 +1396,7 @@ extern int dpkg_main(int argc, char **argv)
                        configure_package(deb_file[i]);
                }
        }
+
        write_status_file(deb_file);
 
        for (i = 0; i < NAME_HASH_PRIME; i++) {
index 7f4dcbf01b1250183c52fc1c857b721e105258af..a933c69483010a221ea82e2f43c96a7d30e64dfb 100644 (file)
@@ -110,12 +110,17 @@ extern int dpkg_deb_main(int argc, char **argv)
        }
        else if (arg_type == arg_type_field) {
                char *field = NULL;
+               char *name;
+               char *value;
                int field_start = 0;
 
-               while ((field = read_package_field(&output_buffer[field_start])) != NULL) {
-                       field_start += (strlen(field) + 1);
-                       if (strstr(field, argv[optind + 1]) == field) {
-                               puts(field + strlen(argv[optind + 1]) + 2);
+               while (1) {
+                       field_start += read_package_field(&output_buffer[field_start], &name, &value);
+                       if (name == NULL) {
+                               break;
+                       }
+                       if (strcmp(name, argv[optind + 1]) == 0) {
+                               puts(value);
                        }
                        free(field);
                }
diff --git a/dpkg.c b/dpkg.c
index e9afe5f6e66f53328929f083c71b9199cbef7eb9..f207b23ac19356a908357aea6e6d67ac94d1fa12 100644 (file)
--- a/dpkg.c
+++ b/dpkg.c
@@ -499,34 +499,17 @@ unsigned int fill_package_struct(char *control_buffer)
 {
        common_node_t *new_node = (common_node_t *) xcalloc(1, sizeof(common_node_t));
 
-       char *field;
-       char *field_name;
-       char *field_value;
+       char *field_name = xmalloc(sizeof(char *));
+       char *field_value = xmalloc(sizeof(char *));
        int field_start = 0;
-       int field_length;
-       int seperator_offset;
        int num = -1;
        int buffer_length = strlen(control_buffer);
 
        new_node->version = search_name_hashtable("unknown");
        while (field_start < buffer_length) {
-               field = read_package_field(&control_buffer[field_start]);
-
-               /* Setup start point for next field */
-               field_length = strlen(field);
-               field_start += (field_length + 1);
-
-               seperator_offset = strcspn(field, ":");
-               if (seperator_offset == 0) {
-                       free(field);
-                       continue;
-               }
-               field_name = xstrndup(field, seperator_offset);
-               field_value = field + seperator_offset + 1;
-               field_value += strspn(field_value, " \n\t");
+               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
 
-               /* Should be able to replace this strlen with pointer arithmatic */
-               if (strlen(field_value) == 0) {
+               if (field_name == NULL) {
                        goto fill_package_struct_cleanup; // Oh no, the dreaded goto statement !!
                }
 
@@ -562,7 +545,7 @@ unsigned int fill_package_struct(char *control_buffer)
                }
 fill_package_struct_cleanup:
                free(field_name);
-               free(field);
+               free(field_value);
        }
        if (new_node->version == search_name_hashtable("unknown")) {
                free_package(new_node);
@@ -739,6 +722,23 @@ char *get_depends_field(common_node_t *package, const int depends_type)
        return(depends);
 }
 
+void write_buffer_no_status(FILE *new_status_file, const char *control_buffer)
+{
+       char *name;
+       char *value;
+       int start = 0;
+       while (1) {
+               start += read_package_field(&control_buffer[start], &name, &value);
+               if (name == NULL) {
+                       break;
+               }
+               if (strcmp(name, "Status") != 0) {
+                       fprintf(new_status_file, "%s: %s\n", name, value);
+               }
+       }
+       return;
+}
+
 /* This could do with a cleanup */
 void write_status_file(deb_file_t **deb_file)
 {
@@ -748,11 +748,8 @@ void write_status_file(deb_file_t **deb_file)
        char *status_from_file;
        char *control_buffer = NULL;
        char *tmp_string;
-       char *field;
        int status_num;
-       int field_length;
        int field_start = 0;
-       int buffer_length;
        int write_flag;
        int i = 0;
 
@@ -762,7 +759,6 @@ void write_status_file(deb_file_t **deb_file)
                tmp_string += strspn(tmp_string, " \n\t");
                package_name = xstrndup(tmp_string, strcspn(tmp_string, "\n\0"));
                write_flag = FALSE;
-
                tmp_string = strstr(control_buffer, "Status:");
                if (tmp_string != NULL) {
                        /* Seperate the status value from the control buffer */
@@ -786,20 +782,11 @@ void write_status_file(deb_file_t **deb_file)
                                        i = 0;
                                        while(deb_file[i] != NULL) {
                                                if (strcmp(package_name, name_hashtable[package_hashtable[deb_file[i]->package]->name]) == 0) {
-                                                       char *last_char;
                                                        /* Write a status file entry with a modified status */
                                                        /* remove trailing \n's */
-                                                       while(1) {
-                                                               last_char = last_char_is(deb_file[i]->control_file, '\n');
-                                                               if (last_char) {
-                                                                       *last_char = '\0';
-                                                               } else {
-                                                                       break;
-                                                               }
-                                                       }
-                                                       fputs(deb_file[i]->control_file, new_status_file);
+                                                       write_buffer_no_status(new_status_file, deb_file[i]->control_file);
                                                        set_status(status_num, "ok", 2);
-                                                       fprintf(new_status_file, "\nStatus: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
+                                                       fprintf(new_status_file, "Status: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
                                                        write_flag = TRUE;
                                                        break;
                                                }
@@ -814,16 +801,17 @@ void write_status_file(deb_file_t **deb_file)
                                        /* Only write the Package, Status, Priority and Section lines */
                                        fprintf(new_status_file, "Package: %s\n", package_name);
                                        fprintf(new_status_file, "Status: %s\n", status_from_hashtable);
-                                       buffer_length = strlen(control_buffer);
-                                       while (field_start < buffer_length) {
-                                               field = read_package_field(&control_buffer[field_start]);
-                                               field_length = strlen(field);
-                                               field_start += (field_length + 1);
-                                               if (strncmp(field, "Priority:", 9) == 0) {
-                                                       fprintf(new_status_file, "Priority:%s\n", field + 9);
+
+                                       while (1) {
+                                               char *field_name;
+                                               char *field_value;
+                                               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
+                                               if (field_name == NULL) {
+                                                       break;
                                                }
-                                               if (strncmp(field, "Section:", 8) == 0) {
-                                                       fprintf(new_status_file, "Section:%s\n", field + 8);
+                                               if ((strcmp(field_name, "Priority") == 0) ||
+                                                       (strcmp(field_name, "Section") == 0)) {
+                                                       fprintf(new_status_file, "%s: %s\n", field_name, field_value);
                                                }
                                        }
                                        write_flag = TRUE;
@@ -831,25 +819,26 @@ void write_status_file(deb_file_t **deb_file)
                                }
                                else if (strcmp("config-files", name_hashtable[state_status]) == 0) {
                                        /* only change the status line */
-                                       buffer_length = strlen(control_buffer);
-                                       while (field_start < buffer_length) {
-                                               field = read_package_field(&control_buffer[field_start]);
+//                                     buffer_length = strlen(control_buffer);
+                                       while (1) {
+                                               char *field_name;
+                                               char *field_value;
+                                               field_start += read_package_field(&control_buffer[field_start], &field_name, &field_value);
+                                               if (field_name == NULL) {
+                                                       break;
+                                               }
                                                /* Setup start point for next field */
-                                               field_length = strlen(field);
-                                               field_start += (field_length + 1);
-                                               if (strncmp(field, "Status:", 7) == 0) {
+                                               if (strcmp(field_name, "Status") == 0) {
                                                        fprintf(new_status_file, "Status: %s\n", status_from_hashtable);
                                                } else {
-                                                       fprintf(new_status_file, "%s\n", field);
+                                                       fprintf(new_status_file, "%s: %s\n", field_name, field_value);
                                                }
-                                               free(field);
                                        }
                                        write_flag = TRUE;
                                        fputs("\n", new_status_file);
                                }
                        }
                }
-
                /* If the package from the status file wasnt handle above, do it now*/
                if (write_flag == FALSE) {
                        fprintf(new_status_file, "%s\n\n", control_buffer);
@@ -866,20 +855,9 @@ void write_status_file(deb_file_t **deb_file)
        for(i = 0; deb_file[i] != NULL; i++) {
                status_num = search_status_hashtable(name_hashtable[package_hashtable[deb_file[i]->package]->name]);
                if (strcmp("reinstreq", name_hashtable[get_status(status_num, 2)]) == 0) {
-                       char *last_char;
-                       /* remove trailing \n's */
-                       while(1) {
-                               last_char = last_char_is(deb_file[i]->control_file, '\n');
-                               if (last_char) {
-                                       *last_char = '\0';
-                               } else {
-                                       break;
-                               }
-                       }
-                               
-                       fputs(deb_file[i]->control_file, new_status_file);
+                       write_buffer_no_status(new_status_file, deb_file[i]->control_file);
                        set_status(status_num, "ok", 2);
-                       fprintf(new_status_file, "\nStatus: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
+                       fprintf(new_status_file, "Status: %s\n\n", name_hashtable[status_hashtable[status_num]->status]);
                }
        }
        fclose(old_status_file);
@@ -1037,9 +1015,11 @@ char **create_list(const char *filename)
        int length = 0;
        int count = 0;
 
-       list_stream = xfopen(filename, "r");
+       /* dont use [xw]fopen here, handle error ourself */
+       list_stream = fopen(filename, "r");
        if (list_stream == NULL) {
-               return(NULL);
+               *file_list = NULL;
+               return(file_list);
        }
        while (getline(&line, &length, list_stream) != -1) {
                file_list = xrealloc(file_list, sizeof(char *) * (length + 1));
@@ -1053,6 +1033,7 @@ char **create_list(const char *filename)
                length = 0;
        }
        fclose(list_stream);
+
        if (count == 0) {
                return(NULL);
        } else {
@@ -1212,6 +1193,7 @@ void purge_package(const unsigned int package_num)
        if (run_package_script(package_name, "postrm") == -1) {
                error_msg_and_die("postrm fialure.. set status to what?");
        }
+
        /* Change package status */
        set_status(status_num, "purge", 1);
        set_status(status_num, "not-installed", 3);
@@ -1219,7 +1201,6 @@ void purge_package(const unsigned int package_num)
 
 void unpack_package(deb_file_t *deb_file)
 {
-//     const unsigned int package_name_num = package_hashtable[deb_file->package]->name;
        const char *package_name = name_hashtable[package_hashtable[deb_file->package]->name];
        const unsigned int status_num = search_status_hashtable(package_name);
        const unsigned int status_package_num = status_hashtable[status_num]->status;
@@ -1248,7 +1229,6 @@ void unpack_package(deb_file_t *deb_file)
 
        /* Create the list file */
        strcat(info_prefix, "list");
-
        out_stream = xfopen(info_prefix, "w");                  
        deb_extract(deb_file->filename, out_stream, (extract_quiet | extract_data_tar_gz | extract_list), NULL, NULL);
        fclose(out_stream);
@@ -1393,7 +1373,7 @@ extern int dpkg_main(int argc, char **argv)
        /* TODO: check dependencies before removing */
        if ((dpkg_opt & dpkg_opt_force_ignore_depends) != dpkg_opt_force_ignore_depends) {
                if (!check_deps(deb_file, 0, deb_count)) {
-                       error_msg_and_die("Dependency check fialed");
+                       error_msg_and_die("Dependency check failed");
                }
        }
 
@@ -1416,6 +1396,7 @@ extern int dpkg_main(int argc, char **argv)
                        configure_package(deb_file[i]);
                }
        }
+
        write_status_file(deb_file);
 
        for (i = 0; i < NAME_HASH_PRIME; i++) {
index 7f4dcbf01b1250183c52fc1c857b721e105258af..a933c69483010a221ea82e2f43c96a7d30e64dfb 100644 (file)
@@ -110,12 +110,17 @@ extern int dpkg_deb_main(int argc, char **argv)
        }
        else if (arg_type == arg_type_field) {
                char *field = NULL;
+               char *name;
+               char *value;
                int field_start = 0;
 
-               while ((field = read_package_field(&output_buffer[field_start])) != NULL) {
-                       field_start += (strlen(field) + 1);
-                       if (strstr(field, argv[optind + 1]) == field) {
-                               puts(field + strlen(argv[optind + 1]) + 2);
+               while (1) {
+                       field_start += read_package_field(&output_buffer[field_start], &name, &value);
+                       if (name == NULL) {
+                               break;
+                       }
+                       if (strcmp(name, argv[optind + 1]) == 0) {
+                               puts(value);
                        }
                        free(field);
                }
index a9c1a870f7f670896804a8daea100ff451178064..bf5f0c1c5236becedce66e34f185aaf213cbaf4d 100644 (file)
@@ -245,7 +245,7 @@ char *unarchive(FILE *src_stream, FILE *out_stream, file_header_t *(*get_header)
        const int extract_function, const char *prefix, char **extract_names);
 char *deb_extract(const char *package_filename, FILE *out_stream, const int extract_function,
        const char *prefix, const char *filename);
-char *read_package_field(const char *package_buffer);
+int read_package_field(const char *package_buffer, char **field_name, char **field_value);
 char *fgets_str(FILE *file, const char *terminating_string);
 
 extern int unzip(FILE *l_in_file, FILE *l_out_file);
index a9c1a870f7f670896804a8daea100ff451178064..bf5f0c1c5236becedce66e34f185aaf213cbaf4d 100644 (file)
@@ -245,7 +245,7 @@ char *unarchive(FILE *src_stream, FILE *out_stream, file_header_t *(*get_header)
        const int extract_function, const char *prefix, char **extract_names);
 char *deb_extract(const char *package_filename, FILE *out_stream, const int extract_function,
        const char *prefix, const char *filename);
-char *read_package_field(const char *package_buffer);
+int read_package_field(const char *package_buffer, char **field_name, char **field_value);
 char *fgets_str(FILE *file, const char *terminating_string);
 
 extern int unzip(FILE *l_in_file, FILE *l_out_file);
index 3715230feab3a003b3766867e1715a1fed605c38..f561df8311b10c600478654cf506bcbb2aeb1a02 100644 (file)
@@ -3,25 +3,89 @@
 #include "libbb.h"
 
 /*
- * Returns a [multi-line] package field
+ * Gets the next package field from package_buffer, seperated into the field name
+ * and field value, it returns the int offset to the first character of the next field
  */
-extern char *read_package_field(const char *package_buffer)
+int read_package_field(const char *package_buffer, char **field_name, char **field_value)
 {
-       int field_length = 0;   
-       int buffer_length = 0;
+       int offset_name_start = 0;
+       int offset_name_end = 0;
+       int offset_value_start = 0;
+       int offset_value_end = 0;
+       int offset = 0;
+       int next_offset;
+       int name_length;
+       int value_length;
+       int exit_flag = FALSE;
 
        if (package_buffer == NULL) {
-               return(NULL);
+               *field_name = NULL;
+               *field_value = NULL;
+               return(-1);
        }
-       buffer_length = strlen(package_buffer);
-       field_length = strcspn(package_buffer, "\n");
-       while (field_length < buffer_length) {
-               if (package_buffer[field_length + 1] != ' ') {
-                       return(xstrndup(package_buffer, field_length));
+       while (1) {
+               next_offset = offset + 1;
+               switch (package_buffer[offset]) {
+                       case('\0'):
+                               exit_flag = TRUE;
+                               break;
+                       case(':'):
+                               if (offset_name_end == 0) {
+                                       offset_name_end = offset;
+                                       offset_value_start = next_offset;
+                               }
+                               /* TODO: Name might still have trailing spaces if ':' isnt
+                                * immediately after name */
+                               break;
+                       case('\n'):
+                               /* TODO: The char next_offset may be out of bounds */
+                               if (package_buffer[next_offset] != ' ') {
+                                       exit_flag = TRUE;
+                                       break;
+                               }
+                       case('\t'):
+                       case(' '):
+                               /* increment the value start point if its a just filler */
+                               if (offset_name_start == offset) {
+                                       offset_name_start++;
+                               }
+                               if (offset_value_start == offset) {
+                                       offset_value_start++;
+                               }
+                               break;
                }
-               field_length++;
-               field_length += strcspn(&package_buffer[field_length], "\n");
+               if (exit_flag == TRUE) {
+                       /* Check that the names are valid */
+                       offset_value_end = offset;
+                       name_length = offset_name_end - offset_name_start;
+                       value_length = offset_value_end - offset_value_start;
+                       if (name_length == 0) {
+                               break;
+                       }
+                       if ((name_length > 0) && (value_length > 0)) {
+                               break;
+                       }
+
+                       /* If not valid, start fresh with next field */
+                       exit_flag = FALSE;
+                       offset_name_start = offset + 1;
+                       offset_name_end = 0;
+                       offset_value_start = offset + 1;
+                       offset_value_end = offset + 1;
+                       offset++;
+               }
+               offset++;
+       }
+       if (name_length == 0) {
+               *field_name = NULL;
+       } else {
+               *field_name = xstrndup(&package_buffer[offset_name_start], name_length);
+       }
+       if (value_length > 0) {
+               *field_value = xstrndup(&package_buffer[offset_value_start], value_length);
+       } else {
+               *field_value = NULL;
        }
-       return(xstrdup(package_buffer));
+       return(next_offset);
 }