summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan McGee <dpmcgee@gmail.com>2012-02-13 13:53:32 -0600
committerAlon Levy <alevy@redhat.com>2012-02-14 10:44:49 +0200
commitaebe837d3ae58f7bbd48bcb010680c1b95f8dc6a (patch)
tree843c50c502ce5090f8adff68176ba568be738a56
parent5868c99da65610898202a5cc896e36e4835b5b87 (diff)
downloadspice-aebe837d3ae58f7bbd48bcb010680c1b95f8dc6a.tar.gz
spice-aebe837d3ae58f7bbd48bcb010680c1b95f8dc6a.tar.xz
spice-aebe837d3ae58f7bbd48bcb010680c1b95f8dc6a.zip
Add casts for compatibility purposes
Some non-Linux platforms return a (caddr_t *) result for the return value of mmap(), which is very unfortunate. Add a (void *) cast to explicitly avoid the warning when compiling with -Werror. For the IO vector related stuff, signed vs. unsigned comes into play so adding a (void *) cast here is technically correct for all platforms. Signed-off-by: Dan McGee <dpmcgee@gmail.com>
-rw-r--r--common/marshaller.c2
-rw-r--r--server/reds.c2
-rw-r--r--tools/reds_stat.c2
3 files changed, 3 insertions, 3 deletions
diff --git a/common/marshaller.c b/common/marshaller.c
index 1446df88..9023d084 100644
--- a/common/marshaller.c
+++ b/common/marshaller.c
@@ -525,7 +525,7 @@ int spice_marshaller_fill_iovec(SpiceMarshaller *m, struct iovec *vec,
if (v == n_vec) {
return v; /* Not enough space in vec */
}
- vec[v].iov_base = item->data + skip_bytes;
+ vec[v].iov_base = (void *)item->data + skip_bytes;
vec[v].iov_len = item->len - skip_bytes;
skip_bytes = 0;
v++;
diff --git a/server/reds.c b/server/reds.c
index 828ba654..ab0dfaff 100644
--- a/server/reds.c
+++ b/server/reds.c
@@ -3623,7 +3623,7 @@ static int do_spice_init(SpiceCoreInterface *core_interface)
if (ftruncate(fd, REDS_STAT_SHM_SIZE) == -1) {
red_error("statistics ftruncate failed, %s", strerror(errno));
}
- reds->stat = mmap(NULL, REDS_STAT_SHM_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
+ reds->stat = (SpiceStat *)mmap(NULL, REDS_STAT_SHM_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
if (reds->stat == (SpiceStat *)MAP_FAILED) {
red_error("statistics mmap failed, %s", strerror(errno));
}
diff --git a/tools/reds_stat.c b/tools/reds_stat.c
index 39d17afe..5e9705c7 100644
--- a/tools/reds_stat.c
+++ b/tools/reds_stat.c
@@ -84,7 +84,7 @@ int main(int argc, char **argv)
return -1;
}
shm_size = sizeof(SpiceStat);
- reds_stat = mmap(NULL, shm_size, PROT_READ, MAP_SHARED, fd, 0);
+ reds_stat = (SpiceStat *)mmap(NULL, shm_size, PROT_READ, MAP_SHARED, fd, 0);
if (reds_stat == (SpiceStat *)MAP_FAILED) {
perror("mmap");
goto error1;