summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Kraai2001-10-18 15:08:30 +0000
committerMatt Kraai2001-10-18 15:08:30 +0000
commit3dd4f5ed69cb6cfc603cae9c79b93eca89d888cc (patch)
tree0d3bef69e4ca2c2992396d073c02b5ed0a51ef45
parenta0f0ae5a7acd71c3c56df5183e38c189c3151c98 (diff)
downloadbusybox-3dd4f5ed69cb6cfc603cae9c79b93eca89d888cc.zip
busybox-3dd4f5ed69cb6cfc603cae9c79b93eca89d888cc.tar.gz
Remove xmalloc calls.
-rw-r--r--archival/dpkg.c59
-rw-r--r--dpkg.c59
2 files changed, 58 insertions, 60 deletions
diff --git a/archival/dpkg.c b/archival/dpkg.c
index a006972..95c8b88 100644
--- a/archival/dpkg.c
+++ b/archival/dpkg.c
@@ -506,60 +506,59 @@ unsigned int fill_package_struct(char *control_buffer)
{
common_node_t *new_node = (common_node_t *) xcalloc(1, sizeof(common_node_t));
- char **field_name = xmalloc(sizeof(char *));
- char **field_value = xmalloc(sizeof(char *));
+ char *field_name;
+ char *field_value;
int field_start = 0;
int num = -1;
int buffer_length = strlen(control_buffer);
new_node->version = search_name_hashtable("unknown");
while (field_start < buffer_length) {
- field_start += read_package_field(&control_buffer[field_start], field_name, field_value);
+ field_start += read_package_field(&control_buffer[field_start],
+ &field_name, &field_value);
- if (*field_name == NULL) {
+ if (field_name == NULL) {
goto fill_package_struct_cleanup; // Oh no, the dreaded goto statement !!
}
- if (strcmp(*field_name, "Package") == 0) {
- new_node->name = search_name_hashtable(*field_value);
+ if (strcmp(field_name, "Package") == 0) {
+ new_node->name = search_name_hashtable(field_value);
}
- else if (strcmp(*field_name, "Version") == 0) {
- new_node->version = search_name_hashtable(*field_value);
+ else if (strcmp(field_name, "Version") == 0) {
+ new_node->version = search_name_hashtable(field_value);
}
- else if (strcmp(*field_name, "Pre-Depends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_PRE_DEPENDS);
+ else if (strcmp(field_name, "Pre-Depends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_PRE_DEPENDS);
}
- else if (strcmp(*field_name, "Depends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_DEPENDS);
+ else if (strcmp(field_name, "Depends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_DEPENDS);
}
- else if (strcmp(*field_name, "Replaces") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_REPLACES);
+ else if (strcmp(field_name, "Replaces") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_REPLACES);
}
- else if (strcmp(*field_name, "Provides") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_PROVIDES);
+ else if (strcmp(field_name, "Provides") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_PROVIDES);
}
- else if (strcmp(*field_name, "Conflicts") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_CONFLICTS);
+ else if (strcmp(field_name, "Conflicts") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_CONFLICTS);
}
- else if (strcmp(*field_name, "Suggests") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_SUGGESTS);
+ else if (strcmp(field_name, "Suggests") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_SUGGESTS);
}
- else if (strcmp(*field_name, "Recommends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_RECOMMENDS);
+ else if (strcmp(field_name, "Recommends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_RECOMMENDS);
}
- else if (strcmp(*field_name, "Enhances") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_ENHANCES);
+ else if (strcmp(field_name, "Enhances") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_ENHANCES);
}
fill_package_struct_cleanup:
- if (*field_name) {
- free(*field_name);
+ if (field_name) {
+ free(field_name);
}
- if (*field_value) {
- free(*field_value);
+ if (field_value) {
+ free(field_value);
}
}
- free(field_name);
- free(field_value);
if (new_node->version == search_name_hashtable("unknown")) {
free_package(new_node);
diff --git a/dpkg.c b/dpkg.c
index a006972..95c8b88 100644
--- a/dpkg.c
+++ b/dpkg.c
@@ -506,60 +506,59 @@ unsigned int fill_package_struct(char *control_buffer)
{
common_node_t *new_node = (common_node_t *) xcalloc(1, sizeof(common_node_t));
- char **field_name = xmalloc(sizeof(char *));
- char **field_value = xmalloc(sizeof(char *));
+ char *field_name;
+ char *field_value;
int field_start = 0;
int num = -1;
int buffer_length = strlen(control_buffer);
new_node->version = search_name_hashtable("unknown");
while (field_start < buffer_length) {
- field_start += read_package_field(&control_buffer[field_start], field_name, field_value);
+ field_start += read_package_field(&control_buffer[field_start],
+ &field_name, &field_value);
- if (*field_name == NULL) {
+ if (field_name == NULL) {
goto fill_package_struct_cleanup; // Oh no, the dreaded goto statement !!
}
- if (strcmp(*field_name, "Package") == 0) {
- new_node->name = search_name_hashtable(*field_value);
+ if (strcmp(field_name, "Package") == 0) {
+ new_node->name = search_name_hashtable(field_value);
}
- else if (strcmp(*field_name, "Version") == 0) {
- new_node->version = search_name_hashtable(*field_value);
+ else if (strcmp(field_name, "Version") == 0) {
+ new_node->version = search_name_hashtable(field_value);
}
- else if (strcmp(*field_name, "Pre-Depends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_PRE_DEPENDS);
+ else if (strcmp(field_name, "Pre-Depends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_PRE_DEPENDS);
}
- else if (strcmp(*field_name, "Depends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_DEPENDS);
+ else if (strcmp(field_name, "Depends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_DEPENDS);
}
- else if (strcmp(*field_name, "Replaces") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_REPLACES);
+ else if (strcmp(field_name, "Replaces") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_REPLACES);
}
- else if (strcmp(*field_name, "Provides") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_PROVIDES);
+ else if (strcmp(field_name, "Provides") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_PROVIDES);
}
- else if (strcmp(*field_name, "Conflicts") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_CONFLICTS);
+ else if (strcmp(field_name, "Conflicts") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_CONFLICTS);
}
- else if (strcmp(*field_name, "Suggests") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_SUGGESTS);
+ else if (strcmp(field_name, "Suggests") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_SUGGESTS);
}
- else if (strcmp(*field_name, "Recommends") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_RECOMMENDS);
+ else if (strcmp(field_name, "Recommends") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_RECOMMENDS);
}
- else if (strcmp(*field_name, "Enhances") == 0) {
- add_split_dependencies(new_node, *field_value, EDGE_ENHANCES);
+ else if (strcmp(field_name, "Enhances") == 0) {
+ add_split_dependencies(new_node, field_value, EDGE_ENHANCES);
}
fill_package_struct_cleanup:
- if (*field_name) {
- free(*field_name);
+ if (field_name) {
+ free(field_name);
}
- if (*field_value) {
- free(*field_value);
+ if (field_value) {
+ free(field_value);
}
}
- free(field_name);
- free(field_value);
if (new_node->version == search_name_hashtable("unknown")) {
free_package(new_node);