diff options
author | Panu Matilainen <pmatilai@redhat.com> | 2009-09-22 12:32:02 +0300 |
---|---|---|
committer | Panu Matilainen <pmatilai@redhat.com> | 2009-11-25 08:54:23 +0200 |
commit | 00628c8999ec713499a03c67c792756bc3c33832 (patch) | |
tree | 513ceca4e8eb14d6e37c01f83fc72ea32cb4356d | |
parent | ebcd87c7e72220c66b137506ca54a3bf960bf900 (diff) | |
download | rpm-00628c8999ec713499a03c67c792756bc3c33832.tar.gz |
Unbreak --tid and --querybynumber
- broken ages ago somewhere between 4.4.x and 4.6.x by typing changes
presumably
- C99 strtoul() can figure the base on its own, remove unnecessary work
(cherry picked from commit eae127b7d835a2cdccd0abad75141efd8cb3a2bf)
-rw-r--r-- | lib/query.c | 44 |
1 files changed, 10 insertions, 34 deletions
diff --git a/lib/query.c b/lib/query.c index 2d777c41d..bcb58d9f6 100644 --- a/lib/query.c +++ b/lib/query.c @@ -427,22 +427,10 @@ int rpmQueryVerify(QVA_t qva, rpmts ts, const char * arg) } break; case RPMQV_TID: - { int mybase = 10; - const char * myarg = arg; - char * end = NULL; - unsigned long iid; - - /* XXX should be in strtoul */ - if (*myarg == '0') { - myarg++; - mybase = 8; - if (*myarg == 'x') { - myarg++; - mybase = 16; - } - } - iid = strtoul(myarg, &end, mybase); - if ((*end) || (end == arg) || (iid == ULONG_MAX)) { + { char * end = NULL; + rpm_tid_t iid = strtoul(arg, &end, 0); + + if ((*end) || (end == arg) || (iid == UINT_MAX)) { rpmlog(RPMLOG_ERR, _("malformed %s: %s\n"), "tid", arg); return 1; } @@ -513,31 +501,19 @@ int rpmQueryVerify(QVA_t qva, rpmts ts, const char * arg) } break; case RPMQV_DBOFFSET: - { int mybase = 10; - const char * myarg = arg; - char * end = NULL; - unsigned long recOffset; - - /* XXX should be in strtoul */ - if (*myarg == '0') { - myarg++; - mybase = 8; - if (*myarg == 'x') { - myarg++; - mybase = 16; - } - } - recOffset = strtoul(myarg, &end, mybase); - if ((*end) || (end == arg) || (recOffset == ULONG_MAX)) { + { char * end = NULL; + unsigned int recOffset = strtoul(arg, &end, 0); + + if ((*end) || (end == arg) || (recOffset == UINT_MAX)) { rpmlog(RPMLOG_ERR, _("invalid package number: %s\n"), arg); return 1; } - rpmlog(RPMLOG_DEBUG, "package record number: %lu\n", recOffset); + rpmlog(RPMLOG_DEBUG, "package record number: %u\n", recOffset); /* RPMDBI_PACKAGES */ qva->qva_mi = rpmtsInitIterator(ts, RPMDBI_PACKAGES, &recOffset, sizeof(recOffset)); if (qva->qva_mi == NULL) { rpmlog(RPMLOG_ERR, - _("record %lu could not be read\n"), recOffset); + _("record %u could not be read\n"), recOffset); res = 1; } else res = rpmcliShowMatches(qva, ts); |