[master] 4371f7b Various code-polishings to make OmnitiOS happy.
Poul-Henning Kamp
phk at varnish-cache.org
Thu Sep 6 14:06:00 CEST 2012
commit 4371f7bd07e63ccb0338ed12c295bfb04968cb8d
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date: Thu Sep 6 12:05:04 2012 +0000
Various code-polishings to make OmnitiOS happy.
Some of them are rather pedantic: "NULL" vs "(char*)0" -- Really ?
diff --git a/bin/varnishd/mgt/mgt_child.c b/bin/varnishd/mgt/mgt_child.c
index 95d1794..e6f8700 100644
--- a/bin/varnishd/mgt/mgt_child.c
+++ b/bin/varnishd/mgt/mgt_child.c
@@ -476,7 +476,7 @@ mgt_sigchld(const struct vev *e, int what)
assert(r == child_pid);
vsb = VSB_new_auto();
XXXAN(vsb);
- VSB_printf(vsb, "Child (%d) %s", r, status ? "died" : "ended");
+ VSB_printf(vsb, "Child (%ld) %s", (long)r, status ? "died" : "ended");
if (WIFEXITED(status) && WEXITSTATUS(status)) {
VSB_printf(vsb, " status=%d", WEXITSTATUS(status));
exit_status |= 0x20;
diff --git a/bin/varnishd/mgt/mgt_vcc.c b/bin/varnishd/mgt/mgt_vcc.c
index f7f199a..d2d077f 100644
--- a/bin/varnishd/mgt/mgt_vcc.c
+++ b/bin/varnishd/mgt/mgt_vcc.c
@@ -177,7 +177,7 @@ static void
run_cc(void *priv)
{
mgt_sandbox(SANDBOX_CC);
- (void)execl("/bin/sh", "/bin/sh", "-c", priv, NULL);
+ (void)execl("/bin/sh", "/bin/sh", "-c", priv, (char*)0);
}
/*--------------------------------------------------------------------
diff --git a/bin/varnishd/storage/storage_persistent.c b/bin/varnishd/storage/storage_persistent.c
index b4e8c8c..5d31caf 100644
--- a/bin/varnishd/storage/storage_persistent.c
+++ b/bin/varnishd/storage/storage_persistent.c
@@ -320,7 +320,7 @@ smp_open(const struct stevedore *st)
/* We trust the parent to give us a valid silo, for good measure: */
AZ(smp_valid_silo(sc));
- AZ(mprotect(sc->base, 4096, PROT_READ));
+ AZ(mprotect((void*)sc->base, 4096, PROT_READ));
sc->ident = SIGN_DATA(&sc->idn);
diff --git a/bin/varnishd/storage/storage_persistent_mgt.c b/bin/varnishd/storage/storage_persistent_mgt.c
index bb0eb38..c5b1dfd 100644
--- a/bin/varnishd/storage/storage_persistent_mgt.c
+++ b/bin/varnishd/storage/storage_persistent_mgt.c
@@ -178,7 +178,7 @@ smp_mgt_init(struct stevedore *parent, int ac, char * const *av)
else
target = NULL;
- sc->base = mmap(target, sc->mediasize, PROT_READ|PROT_WRITE,
+ sc->base = (void*)mmap(target, sc->mediasize, PROT_READ|PROT_WRITE,
MAP_NOCORE | MAP_NOSYNC | MAP_SHARED, sc->fd, 0);
if (sc->base == MAP_FAILED)
diff --git a/bin/varnishtest/vtc_varnish.c b/bin/varnishtest/vtc_varnish.c
index 0ddf1d0..0036720 100644
--- a/bin/varnishtest/vtc_varnish.c
+++ b/bin/varnishtest/vtc_varnish.c
@@ -377,10 +377,10 @@ varnish_launch(struct varnish *v)
AZ(close(v->fds[3]));
for (i = 3; i <getdtablesize(); i++)
(void)close(i);
- AZ(execl("/bin/sh", "/bin/sh", "-c", VSB_data(vsb), NULL));
+ AZ(execl("/bin/sh", "/bin/sh", "-c", VSB_data(vsb), (char*)0));
exit(1);
} else {
- vtc_log(v->vl, 3, "PID: %d", v->pid);
+ vtc_log(v->vl, 3, "PID: %ld", (long)v->pid);
}
AZ(close(v->fds[0]));
AZ(close(v->fds[3]));
More information about the varnish-commit
mailing list