X-Git-Url: https://pd.if.org/git/?a=blobdiff_plain;f=lib%2Ffindpkg.c;h=900b6cbc74ebd51bfb5ce0184d95715793e72c60;hb=b7a4dd34227b6cf84908ffde9f2c4395f29fe1b4;hp=865f407b4f8dc638060c636e3df70da3c7c8003d;hpb=67caae5874fda7371cb543e248b1014ae17e14d7;p=zpackage

diff --git a/lib/findpkg.c b/lib/findpkg.c
index 865f407..900b6cb 100644
--- a/lib/findpkg.c
+++ b/lib/findpkg.c
@@ -9,6 +9,8 @@
 
 #include "sqlite3.h"
 
+#define DMARK fprintf(stderr, "mark %s %s:%d\n", __FILE__, __func__, __LINE__)
+
 int zpm_parse_package(char *pstr, char *name, char *ver, int *rel) {
 	if (name) *name = 0;
 	if (ver) *ver = 0;
@@ -94,7 +96,7 @@ void zpm_sqlite_error(struct zpm *zpm) {
 
 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'",
@@ -106,11 +108,16 @@ 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 = 0;
 	char package[32];
 	char version[32];
 	int release;
 
+	if (pkgstr == NULL) {
+		fprintf(stderr, "trying to parse null string\n");
+		return NULL;
+	}
+
 	/* null pkgstr find "best" package
 	 * best is shortest complete package if any are complete
 	 * shortest incomplete if any are incomplete
@@ -132,9 +139,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);
@@ -152,12 +159,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 = strdup((const char *)sqlite3_column_text(stmt, 0));
+			pkgid = strdup((const char *)sqlite3_column_text(stmt, 0));
 			break;
 		case SQLITE_DONE:
 			/* not found */
@@ -168,5 +179,5 @@ char *zpm_findpkg(struct zpm *zpm, char *pkgstr) {
 	}
 	sqlite3_finalize(stmt);
 
-	return zpm->pkgid;
+	return pkgid;
 }