use stage instead of phase in dbquery

[zpackage] / lib / findpkg.c

diff --git a/lib/findpkg.c b/lib/findpkg.c
index a8d8ff49f791b7010be411f537a2e171b26f8380..6f69876f76915149e2e4d5b276a2477b36d98100 100644 (file)
--- a/lib/findpkg.c
+++ b/lib/findpkg.c
@@ -1,3 +1,5 @@
+#define _POSIX_C_SOURCE 200809L
+
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
@@ -28,6 +30,9 @@ int zpm_parse_package(char *pstr, char *name, char *ver, int *rel) {
 
        /* everything up to the first '-' is in the name */
        while (*pstr) {
+               if (*pstr == '\'' || !isgraph(*pstr)) {
+                       return 0;
+               }
                if (*pstr == '-' && isdigit(*(pstr+1))) {
                        break;
                }
@@ -42,6 +47,9 @@ int zpm_parse_package(char *pstr, char *name, char *ver, int *rel) {
                pstr++;
        }
        while (*pstr && *pstr != '-') {
+               if (*pstr == '\'' || !isgraph(*pstr)) {
+                       return 0;
+               }
                if (ver) {
                        *ver++ = *pstr;
                }
@@ -81,12 +89,12 @@ struct zpm_package *zpm_package_alloc(struct zpm *zpm) {
 void zpm_sqlite_error(struct zpm *zpm) {
        zpm->error = 1;
        if (zpm->errmsg) free(zpm->errmsg);
-       zpm->errmsg = zpm_strdup((char *)sqlite3_errmsg(zpm->db));
+       zpm->errmsg = strdup((const char *)sqlite3_errmsg(zpm->db));
 }
 
 char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
        char *select = "select pkgid, package, version, release from packages_pkgid";
-       char *group = "group by package having max( version||'-'||release collate vercmp) order by length(package), package, version||'-'||release collate vercmp limit 1";
+       char *group = "group by package having max( version||'-'||release collate vercmp) order by length(package), package, version||'-'||release collate vercmp";
 #if 0
        char *sstr[] = {
                "status = 'installed'",
@@ -98,7 +106,7 @@ char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
 //     char *order = "order by package, version collate vercmp, cast(release as integer)";
        sqlite3_str *sql;
        sqlite3_stmt *stmt;
-       char *query;
+       char *query, *pkgid;
        char package[32];
        char version[32];
        int release;
@@ -124,9 +132,9 @@ char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
 
        sql = sqlite3_str_new(zpm->db);
        sqlite3_str_appendall(sql, select);
-       sqlite3_str_appendf(sql, " where package = %q", package);
+       sqlite3_str_appendf(sql, " where package = %Q", package);
        if (*version) {
-               sqlite3_str_appendf(sql, " and version = %q", version);
+               sqlite3_str_appendf(sql, " and version = %Q", version);
        }
        if (release) {
                sqlite3_str_appendf(sql, " and release = %d", release);
@@ -144,12 +152,16 @@ char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
        sqlite3_prepare_v2(zpm->db, query, strlen(query), &stmt, NULL);
        sqlite3_free(query);
 
-       free(zpm->pkgid);
+#if 0
+       if (zpm->pkgid) {
+               free(zpm->pkgid);
+       }
        zpm->pkgid = 0;
+#endif
 
        switch (sqlite3_step(stmt)) {
                case SQLITE_ROW:
-                       zpm->pkgid = zpm_strdup((char *)sqlite3_column_text(stmt, 0));
+                       pkgid = strdup((const char *)sqlite3_column_text(stmt, 0));
                        break;
                case SQLITE_DONE:
                        /* not found */
@@ -160,5 +172,5 @@ char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
        }
        sqlite3_finalize(stmt);
 
-       return zpm->pkgid;
+       return pkgid;
 }