aboutsummaryrefslogtreecommitdiff
path: root/archival
diff options
context:
space:
mode:
authorEric Andersen <andersen@codepoet.org>2006-01-30 18:33:12 +0000
committerEric Andersen <andersen@codepoet.org>2006-01-30 18:33:12 +0000
commit2cdd4d56ffc3b467d5ffa76e3c4cd009dc311097 (patch)
tree39987feb16177a5908fd7965f0589fdcc2874850 /archival
parentd78aea8b8eb1da818c04929a2bddbed01a796c58 (diff)
downloadbusybox-2cdd4d56ffc3b467d5ffa76e3c4cd009dc311097.tar.gz
passing around an int as a void* is a very bad idea
Diffstat (limited to 'archival')
-rw-r--r--archival/rpm.c15
1 files changed, 5 insertions, 10 deletions
diff --git a/archival/rpm.c b/archival/rpm.c
index 88e748550..7f69f0510 100644
--- a/archival/rpm.c
+++ b/archival/rpm.c
@@ -247,18 +247,15 @@ rpm_index **rpm_gettags(int fd, int *num_tags)
int bsearch_rpmtag(const void *key, const void *item)
{
+ int *tag = (int *)key;
rpm_index **tmp = (rpm_index **) item;
- /* gcc throws warnings here when sizeof(void*)!=sizeof(int) ...
- * it's ok to ignore it because this isn't a 'real' pointer */
- return ((int) key - tmp[0]->tag);
+ return (*tag - tmp[0]->tag);
}
int rpm_getcount(int tag)
{
rpm_index **found;
- /* gcc throws warnings here when sizeof(void*)!=sizeof(int) ...
- * it's ok to ignore it because tag won't be used as a pointer */
- found = bsearch((void *) tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
+ found = bsearch(&tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
if (!found) return 0;
else return found[0]->count;
}
@@ -266,9 +263,7 @@ int rpm_getcount(int tag)
char *rpm_getstring(int tag, int itemindex)
{
rpm_index **found;
- /* gcc throws warnings here when sizeof(void*)!=sizeof(int) ...
- * it's ok to ignore it because tag won't be used as a pointer */
- found = bsearch((void *) tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
+ found = bsearch(&tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
if (!found || itemindex >= found[0]->count) return NULL;
if (found[0]->type == RPM_STRING_TYPE || found[0]->type == RPM_I18NSTRING_TYPE || found[0]->type == RPM_STRING_ARRAY_TYPE) {
int n;
@@ -284,7 +279,7 @@ int rpm_getint(int tag, int itemindex)
int n, *tmpint;
/* gcc throws warnings here when sizeof(void*)!=sizeof(int) ...
* it's ok to ignore it because tag won't be used as a pointer */
- found = bsearch((void *) tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
+ found = bsearch(&tag, mytags, tagcount, sizeof(struct rpmtag *), bsearch_rpmtag);
if (!found || itemindex >= found[0]->count) return -1;
tmpint = (int *) (map + found[0]->offset);
if (found[0]->type == RPM_INT32_TYPE) {