pmatilai commented on this pull request.


> +{
+    rpmlog(RPMLOG_ERR, "bdb_ro error reading %s database\n", dbi->dbi_file);
+}
+
+static int bdbro_Open(rpmdb rdb, rpmDbiTagVal rpmtag, dbiIndex * dbip, int 
flags)
+{
+    const char *dbhome = rpmdbHome(rdb);
+    dbiIndex dbi = NULL;
+    char *path;
+
+    if (dbip)
+       *dbip = NULL;
+    if ((rdb->db_mode & O_ACCMODE) != O_RDONLY)
+       return EPERM;
+    if ((dbi = dbiNew(rdb, rpmtag)) == NULL)
+       return 1;

dbiNew() can't actually fail, so this is superfluous.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/980#pullrequestreview-339715339
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to