@dmnks commented on this pull request.


> @@ -470,11 +470,13 @@ static rpmdbMatchIterator initQueryIterator(QVA_t qva, 
> rpmts ts, const char * ar
 
        if (mi == NULL) {
            struct stat sb;
-           if (lstat(fn, &sb) != 0)
+           char * full_fn = rpmGenPath(rpmtsRootDir(ts), fn, NULL);

How about using the simpler `rpmGetPath()` instead? The Gen variant only 
differs in that it also accepts URLs which I don't think we need here :smile: 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2582#pullrequestreview-1547683591
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2582/review/1547683...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to