summaryrefslogtreecommitdiffstats
path: root/libopkg
diff options
context:
space:
mode:
authorgraham.gower <graham.gower@e8e0d7a0-c8d9-11dd-a880-a1081c7ac358>2009-11-12 01:26:26 (EST)
committer graham.gower <graham.gower@e8e0d7a0-c8d9-11dd-a880-a1081c7ac358>2009-11-12 01:26:26 (EST)
commit99be91a0fdb86a6826a0cdcf8bede14b7d01c9a8 (patch)
treef4936cb82a62d52baf91e0ae62df51c79cf48f28 /libopkg
parent377c695695365f89f0a47507d17562be67d8ca04 (diff)
Cleanup the rest of pkg_parse.c. Give some functions more appropriate names too.
git-svn-id: http://opkg.googlecode.com/svn/trunk@289 e8e0d7a0-c8d9-11dd-a880-a1081c7ac358
Diffstat (limited to 'libopkg')
-rw-r--r--libopkg/opkg_cmd.c4
-rw-r--r--libopkg/opkg_utils.c2
-rw-r--r--libopkg/opkg_utils.h2
-rw-r--r--libopkg/pkg_depends.c6
-rw-r--r--libopkg/pkg_parse.c319
-rw-r--r--libopkg/pkg_parse.h2
6 files changed, 172 insertions, 163 deletions
diff --git a/libopkg/opkg_cmd.c b/libopkg/opkg_cmd.c
index f67a264..afa9ccf 100644
--- a/libopkg/opkg_cmd.c
+++ b/libopkg/opkg_cmd.c
@@ -1350,8 +1350,8 @@ static int opkg_compare_versions_cmd(opkg_conf_t *conf, int argc, char **argv)
if (argc == 3) {
/* this is a bit gross */
struct pkg p1, p2;
- parseVersion(&p1, argv[0]);
- parseVersion(&p2, argv[2]);
+ parse_version(&p1, argv[0]);
+ parse_version(&p2, argv[2]);
return pkg_version_satisfied(&p1, &p2, argv[1]);
} else {
opkg_message(conf, OPKG_ERROR,
diff --git a/libopkg/opkg_utils.c b/libopkg/opkg_utils.c
index d40999f..2bb7c2e 100644
--- a/libopkg/opkg_utils.c
+++ b/libopkg/opkg_utils.c
@@ -48,7 +48,7 @@ long unsigned int get_available_blocks(char * filesystem)
}
/* something to remove whitespace, a hash pooper */
-char *trim_alloc(const char *src)
+char *trim_xstrdup(const char *src)
{
const char *end;
diff --git a/libopkg/opkg_utils.h b/libopkg/opkg_utils.h
index faa504b..7cb0693 100644
--- a/libopkg/opkg_utils.h
+++ b/libopkg/opkg_utils.h
@@ -26,7 +26,7 @@ void free_error_list(void);
void print_error_list(void);
long unsigned int get_available_blocks(char * filesystem);
-char *trim_alloc(const char *line);
+char *trim_xstrdup(const char *line);
int line_is_blank(const char *line);
#endif
diff --git a/libopkg/pkg_depends.c b/libopkg/pkg_depends.c
index e059a25..d93ea0c 100644
--- a/libopkg/pkg_depends.c
+++ b/libopkg/pkg_depends.c
@@ -356,7 +356,7 @@ int version_constraints_satisfied(depend_t * depends, pkg_t * pkg)
temp = pkg_new();
- parseVersion(temp, depends->version);
+ parse_version(temp, depends->version);
comparison = pkg_compare_versions(pkg, temp);
@@ -770,7 +770,7 @@ static int parseDepends(compound_depend_t *compound_depend,
(*src != '|'))
*dest++ = *src++;
*dest = '\0';
- pkg_name = trim_alloc(buffer);
+ pkg_name = trim_xstrdup(buffer);
if (pkg_name == NULL )
return -ENOMEM;
@@ -822,7 +822,7 @@ static int parseDepends(compound_depend_t *compound_depend,
*dest++ = *src++;
*dest = '\0';
- possibilities[i]->version = trim_alloc(buffer);
+ possibilities[i]->version = trim_xstrdup(buffer);
/* fprintf(stderr, "let's print the depends version string:");
fprintf(stderr, "version %s\n", possibilities[i]->version);*/
if (possibilities[i]->version == NULL )
diff --git a/libopkg/pkg_parse.c b/libopkg/pkg_parse.c
index 4f7fcf2..fb479c3 100644
--- a/libopkg/pkg_parse.c
+++ b/libopkg/pkg_parse.c
@@ -24,66 +24,84 @@
#include "pkg_parse.h"
#include "libbb/libbb.h"
-static int isGenericFieldType(char * type, const char * line)
+static int
+is_field(char *type, const char *line)
{
- if(!strncmp(line, type, strlen(type)))
- return 1;
- return 0;
+ if (!strncmp(line, type, strlen(type)))
+ return 1;
+ return 0;
}
-static char * parseGenericFieldType(char * type, const char * raw)
+static char *
+parse_simple(char *type, const char *line)
{
- const char * field_value = raw + (strlen(type) + 1);
- return trim_alloc(field_value);
+ return trim_xstrdup(line + strlen(type) + 1);
}
-static void parseStatus(pkg_t *pkg, const char * raw)
+/*
+ * Parse a comma separated string into an array.
+ */
+static char **
+parse_comma_separated(const char *raw, int *count)
{
- char sw_str[64], sf_str[64], ss_str[64];
+ char **depends = NULL;
+ const char *start, *end;
+ int line_count = 0;
+
+ /* skip past the "Field:" marker */
+ while (*raw && *raw != ':')
+ raw++;
+ raw++;
+
+ if (line_is_blank(raw)) {
+ *count = line_count;
+ return NULL;
+ }
- sscanf(raw, "Status: %s %s %s", sw_str, sf_str, ss_str);
- pkg->state_want = pkg_state_want_from_str(sw_str);
- pkg->state_flag = pkg_state_flag_from_str(sf_str);
- pkg->state_status = pkg_state_status_from_str(ss_str);
+ while (*raw) {
+ depends = xrealloc(depends, sizeof(char *) * (line_count + 1));
+
+ while (isspace(*raw))
+ raw++;
+
+ start = raw;
+ while (*raw != ',' && *raw)
+ raw++;
+ end = raw;
+
+ while (end > start && isspace(*end))
+ end--;
+
+ depends[line_count] = xstrndup(start, end-start);
+
+ line_count++;
+ if (*raw == ',')
+ raw++;
+ }
+
+ *count = line_count;
+ return depends;
}
-static char ** parseDependsString(const char * raw, int * depends_count)
+static void
+parse_status(pkg_t *pkg, const char *sstr)
{
- char ** depends = NULL;
- int line_count = 0;
- char buff[2048], * dest;
+ char sw_str[64], sf_str[64], ss_str[64];
- while(raw && *raw && !isspace(*raw)) {
- raw++;
- }
-
- if(line_is_blank(raw)){
- *depends_count = line_count;
- return NULL;
- }
- while(raw && *raw){
- depends = xrealloc(depends, sizeof(char *) * (line_count + 1));
-
- while(isspace(*raw)) raw++;
-
- dest = buff;
- while((*raw != ',') && *raw)
- *dest++ = *raw++;
-
- *dest = '\0';
- depends[line_count] = trim_alloc(buff);
- if(depends[line_count] ==NULL)
- return NULL;
- line_count++;
- if(*raw == ',')
- raw++;
- }
- *depends_count = line_count;
- return depends;
+ if (sscanf(sstr, "Status: %63s %63s %63s",
+ sw_str, sf_str, ss_str) != 3) {
+ fprintf(stderr, "%s: failed to parse Conffiles line for %s\n",
+ __FUNCTION__, pkg->name);
+ return;
+ }
+
+ pkg->state_want = pkg_state_want_from_str(sw_str);
+ pkg->state_flag = pkg_state_flag_from_str(sf_str);
+ pkg->state_status = pkg_state_status_from_str(ss_str);
}
static void
-parseConffiles(pkg_t *pkg, const char *cstr)
+parse_conffiles(pkg_t *pkg, const char *cstr)
{
char file_name[1024], md5sum[35];
@@ -97,7 +115,7 @@ parseConffiles(pkg_t *pkg, const char *cstr)
}
int
-parseVersion(pkg_t *pkg, const char *vstr)
+parse_version(pkg_t *pkg, const char *vstr)
{
char *colon;
@@ -137,145 +155,136 @@ pkg_parse_line(pkg_t *pkg, const char *line, uint mask)
switch (*line) {
case 'A':
- if((mask & PFM_ARCHITECTURE ) && isGenericFieldType("Architecture:", line))
- pkg->architecture = parseGenericFieldType("Architecture", line);
- else if((mask & PFM_AUTO_INSTALLED) && isGenericFieldType("Auto-Installed:", line)) {
- char *auto_installed_value;
- auto_installed_value = parseGenericFieldType("Auto-Installed:", line);
- if (strcmp(auto_installed_value, "yes") == 0) {
- pkg->auto_installed = 1;
+ if ((mask & PFM_ARCHITECTURE ) && is_field("Architecture", line))
+ pkg->architecture = parse_simple("Architecture", line);
+ else if ((mask & PFM_AUTO_INSTALLED) && is_field("Auto-Installed", line)) {
+ char *tmp = parse_simple("Auto-Installed", line);
+ if (strcmp(tmp, "yes") == 0)
+ pkg->auto_installed = 1;
+ free(tmp);
}
- free(auto_installed_value);
- }
- break;
+ break;
case 'C':
- if((mask & PFM_CONFFILES) && isGenericFieldType("Conffiles", line)){
- reading_conffiles = 1;
- reading_description = 0;
- goto dont_reset_flags;
- }
- else if((mask & PFM_CONFLICTS) && isGenericFieldType("Conflicts", line))
- pkg->conflicts_str = parseDependsString(line, &pkg->conflicts_count);
- break;
+ if ((mask & PFM_CONFFILES) && is_field("Conffiles", line)) {
+ reading_conffiles = 1;
+ reading_description = 0;
+ goto dont_reset_flags;
+ }
+ else if ((mask & PFM_CONFLICTS) && is_field("Conflicts", line))
+ pkg->conflicts_str = parse_comma_separated(line, &pkg->conflicts_count);
+ break;
case 'D':
- if((mask & PFM_DESCRIPTION) && isGenericFieldType("Description", line)) {
- pkg->description = parseGenericFieldType("Description", line);
- reading_conffiles = 0;
- reading_description = 1;
- goto dont_reset_flags;
- }
- else if((mask & PFM_DEPENDS) && isGenericFieldType("Depends", line))
- pkg->depends_str = parseDependsString(line, &pkg->depends_count);
- break;
+ if ((mask & PFM_DESCRIPTION) && is_field("Description", line)) {
+ pkg->description = parse_simple("Description", line);
+ reading_conffiles = 0;
+ reading_description = 1;
+ goto dont_reset_flags;
+ } else if ((mask & PFM_DEPENDS) && is_field("Depends", line))
+ pkg->depends_str = parse_comma_separated(line, &pkg->depends_count);
+ break;
case 'E':
- if((mask & PFM_ESSENTIAL) && isGenericFieldType("Essential:", line)) {
- char *essential_value;
- essential_value = parseGenericFieldType("Essential", line);
- if (strcmp(essential_value, "yes") == 0) {
- pkg->essential = 1;
+ if((mask & PFM_ESSENTIAL) && is_field("Essential", line)) {
+ char *tmp = parse_simple("Essential", line);
+ if (strcmp(tmp, "yes") == 0)
+ pkg->essential = 1;
+ free(tmp);
}
- free(essential_value);
- }
- break;
+ break;
case 'F':
- if((mask & PFM_FILENAME) && isGenericFieldType("Filename:", line))
- pkg->filename = parseGenericFieldType("Filename", line);
- break;
+ if((mask & PFM_FILENAME) && is_field("Filename", line))
+ pkg->filename = parse_simple("Filename", line);
+ break;
case 'I':
- if((mask && PFM_INSTALLED_SIZE) && isGenericFieldType("Installed-Size:", line))
- pkg->installed_size = parseGenericFieldType("Installed-Size", line);
- else if((mask && PFM_INSTALLED_TIME) && isGenericFieldType("Installed-Time:", line)) {
- char *time_str = parseGenericFieldType("Installed-Time", line);
- pkg->installed_time = strtoul(time_str, NULL, 0);
- free (time_str);
- }
- break;
+ if ((mask && PFM_INSTALLED_SIZE) && is_field("Installed-Size", line))
+ pkg->installed_size = parse_simple("Installed-Size", line);
+ else if ((mask && PFM_INSTALLED_TIME) && is_field("Installed-Time", line)) {
+ char *tmp = parse_simple("Installed-Time", line);
+ pkg->installed_time = strtoul(tmp, NULL, 0);
+ free (tmp);
+ }
+ break;
case 'M':
- if(mask && PFM_MD5SUM) {
- if (isGenericFieldType("MD5sum:", line))
- pkg->md5sum = parseGenericFieldType("MD5sum", line);
- /* The old opkg wrote out status files with the wrong
- * case for MD5sum, let's parse it either way */
- else if(isGenericFieldType("MD5Sum:", line))
- pkg->md5sum = parseGenericFieldType("MD5Sum", line);
- } else if((mask & PFM_MAINTAINER) && isGenericFieldType("Maintainer", line))
- pkg->maintainer = parseGenericFieldType("Maintainer", line);
- break;
+ if (mask && PFM_MD5SUM) {
+ if (is_field("MD5sum:", line))
+ pkg->md5sum = parse_simple("MD5sum", line);
+ /* The old opkg wrote out status files with the wrong
+ * case for MD5sum, let's parse it either way */
+ else if (is_field("MD5Sum:", line))
+ pkg->md5sum = parse_simple("MD5Sum", line);
+ } else if((mask & PFM_MAINTAINER) && is_field("Maintainer", line))
+ pkg->maintainer = parse_simple("Maintainer", line);
+ break;
case 'P':
- if((mask & PFM_PACKAGE) && isGenericFieldType("Package:", line))
- pkg->name = parseGenericFieldType("Package", line);
- else if((mask & PFM_PRIORITY) && isGenericFieldType("Priority:", line))
- pkg->priority = parseGenericFieldType("Priority", line);
- else if((mask & PFM_PROVIDES) && isGenericFieldType("Provides", line)){
- pkg->provides_str = parseDependsString(line, &pkg->provides_count);
- }
- else if((mask & PFM_PRE_DEPENDS) && isGenericFieldType("Pre-Depends", line))
- pkg->pre_depends_str = parseDependsString(line, &pkg->pre_depends_count);
- break;
+ if ((mask & PFM_PACKAGE) && is_field("Package", line))
+ pkg->name = parse_simple("Package", line);
+ else if ((mask & PFM_PRIORITY) && is_field("Priority", line))
+ pkg->priority = parse_simple("Priority", line);
+ else if ((mask & PFM_PROVIDES) && is_field("Provides", line))
+ pkg->provides_str = parse_comma_separated(line, &pkg->provides_count);
+ else if ((mask & PFM_PRE_DEPENDS) && is_field("Pre-Depends", line))
+ pkg->pre_depends_str = parse_comma_separated(line, &pkg->pre_depends_count);
+ break;
case 'R':
- if((mask & PFM_RECOMMENDS) && isGenericFieldType("Recommends", line))
- pkg->recommends_str = parseDependsString(line, &pkg->recommends_count);
- else if((mask & PFM_REPLACES) && isGenericFieldType("Replaces", line))
- pkg->replaces_str = parseDependsString(line, &pkg->replaces_count);
-
- break;
+ if ((mask & PFM_RECOMMENDS) && is_field("Recommends", line))
+ pkg->recommends_str = parse_comma_separated(line, &pkg->recommends_count);
+ else if ((mask & PFM_REPLACES) && is_field("Replaces", line))
+ pkg->replaces_str = parse_comma_separated(line, &pkg->replaces_count);
+
+ break;
case 'S':
- if((mask & PFM_SECTION) && isGenericFieldType("Section:", line))
- pkg->section = parseGenericFieldType("Section", line);
+ if ((mask & PFM_SECTION) && is_field("Section", line))
+ pkg->section = parse_simple("Section", line);
#ifdef HAVE_SHA256
- else if((mask & PFM_SHA256SUM) && isGenericFieldType("SHA256sum:", line))
- pkg->sha256sum = parseGenericFieldType("SHA256sum", line);
+ else if ((mask & PFM_SHA256SUM) && is_field("SHA256sum", line))
+ pkg->sha256sum = parse_simple("SHA256sum", line);
#endif
- else if((mask & PFM_SIZE) && isGenericFieldType("Size:", line))
- pkg->size = parseGenericFieldType("Size", line);
- else if((mask & PFM_SOURCE) && isGenericFieldType("Source:", line))
- pkg->source = parseGenericFieldType("Source", line);
- else if((mask & PFM_STATUS) && isGenericFieldType("Status", line))
- parseStatus(pkg, line);
- else if((mask & PFM_SUGGESTS) && isGenericFieldType("Suggests", line))
- pkg->suggests_str = parseDependsString(line, &pkg->suggests_count);
- break;
+ else if ((mask & PFM_SIZE) && is_field("Size", line))
+ pkg->size = parse_simple("Size", line);
+ else if ((mask & PFM_SOURCE) && is_field("Source", line))
+ pkg->source = parse_simple("Source", line);
+ else if ((mask & PFM_STATUS) && is_field("Status", line))
+ parse_status(pkg, line);
+ else if ((mask & PFM_SUGGESTS) && is_field("Suggests", line))
+ pkg->suggests_str = parse_comma_separated(line, &pkg->suggests_count);
+ break;
case 'T':
- if((mask & PFM_TAGS) && isGenericFieldType("Tags:", line))
- pkg->tags = parseGenericFieldType("Tags", line);
- break;
+ if ((mask & PFM_TAGS) && is_field("Tags", line))
+ pkg->tags = parse_simple("Tags", line);
+ break;
case 'V':
- if((mask & PFM_VERSION) && isGenericFieldType("Version", line))
- parseVersion(pkg, line);
- break;
+ if ((mask & PFM_VERSION) && is_field("Version", line))
+ parse_version(pkg, line);
+ break;
case ' ':
- if((mask & PFM_DESCRIPTION) && reading_description) {
- /* we already know it's not blank, so the rest of description */
- pkg->description = xrealloc(pkg->description,
- strlen(pkg->description)
- + 1 + strlen(line) + 1);
- strcat(pkg->description, "\n");
- strcat(pkg->description, (line));
- goto dont_reset_flags;
- }
- else if((mask && PFM_CONFFILES) && reading_conffiles) {
- parseConffiles(pkg, line);
- goto dont_reset_flags;
- }
- break;
+ if ((mask & PFM_DESCRIPTION) && reading_description) {
+ pkg->description = xrealloc(pkg->description,
+ strlen(pkg->description)
+ + 1 + strlen(line) + 1);
+ strcat(pkg->description, "\n");
+ strcat(pkg->description, (line));
+ goto dont_reset_flags;
+ } else if ((mask && PFM_CONFFILES) && reading_conffiles) {
+ parse_conffiles(pkg, line);
+ goto dont_reset_flags;
+ }
+ break;
default:
- /* For package lists, signifies end of package. */
- if(line_is_blank(line)) {
- return 1;
- }
+ /* For package lists, signifies end of package. */
+ if(line_is_blank(line))
+ return 1;
}
reading_description = 0;
diff --git a/libopkg/pkg_parse.h b/libopkg/pkg_parse.h
index 5461607..24f0238 100644
--- a/libopkg/pkg_parse.h
+++ b/libopkg/pkg_parse.h
@@ -18,7 +18,7 @@
#ifndef PKG_PARSE_H
#define PKG_PARSE_H
-int parseVersion(pkg_t *pkg, const char *raw);
+int parse_version(pkg_t *pkg, const char *raw);
int pkg_parse_from_stream(pkg_t *pkg, FILE *fp, uint mask);
int pkg_parse_from_stream_nomalloc(pkg_t *pkg, FILE *fp, uint mask,
char **buf0, size_t buf0len);