[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 10/10] Darwin: Fix compilation warning regarding the
From: |
Andreas Färber |
Subject: |
[Qemu-devel] [PATCH 10/10] Darwin: Fix compilation warning regarding the deprecated daemon() function |
Date: |
Tue, 14 Jun 2011 03:22:07 +0200 |
From: Alexandre Raymond <address@hidden>
Changes since v1: create a wrapper function named qemu_daemon() in oslib-posix.c
instead of putting the OS specific workaround in qemu-nbd.c directly.
On OSX >= 10.5, daemon() is deprecated, resulting in the following warning:
----8<----
qemu-nbd.c: In function ‘main’:
qemu-nbd.c:371: warning: ‘daemon’ is deprecated (declared at
/usr/include/stdlib.h:289)
----8<----
The following trick, used in mDNSResponder, takes care of this warning:
http://www.opensource.apple.com/source/mDNSResponder/mDNSResponder-258.18/mDNSPosix/PosixDaemon.c
On OSX, it temporarily renames the daemon() function before including stdlib.h
and declares it manually as an extern function. This way, the compiler does not
see the declaration from stdlib.h and thus does not display the warning.
Signed-off-by: Alexandre Raymond <address@hidden>
Cc: Blue Swirl <address@hidden>
Signed-off-by: Andreas Färber <address@hidden>
---
osdep.h | 1 +
oslib-posix.c | 16 ++++++++++++++++
qemu-nbd.c | 2 +-
3 files changed, 18 insertions(+), 1 deletions(-)
diff --git a/osdep.h b/osdep.h
index 970d767..6eb9a49 100644
--- a/osdep.h
+++ b/osdep.h
@@ -88,6 +88,7 @@
# define QEMU_GNUC_PREREQ(maj, min) 0
#endif
+int qemu_daemon(int nochdir, int noclose);
void *qemu_memalign(size_t alignment, size_t size);
void *qemu_vmalloc(size_t size);
void qemu_vfree(void *ptr);
diff --git a/oslib-posix.c b/oslib-posix.c
index 7bc5f7c..3a18e86 100644
--- a/oslib-posix.c
+++ b/oslib-posix.c
@@ -26,11 +26,27 @@
* THE SOFTWARE.
*/
+/* The following block of code temporarily renames the daemon() function so the
+ compiler does not see the warning associated with it in stdlib.h on OSX */
+#ifdef __APPLE__
+#define daemon qemu_fake_daemon_function
+#include <stdlib.h>
+#undef daemon
+extern int daemon(int, int);
+#endif
+
#include "config-host.h"
#include "sysemu.h"
#include "trace.h"
#include "qemu_socket.h"
+
+
+int qemu_daemon(int nochdir, int noclose)
+{
+ return daemon(nochdir, noclose);
+}
+
void *qemu_oom_check(void *ptr)
{
if (ptr == NULL) {
diff --git a/qemu-nbd.c b/qemu-nbd.c
index 110d78e..d91c02c 100644
--- a/qemu-nbd.c
+++ b/qemu-nbd.c
@@ -359,7 +359,7 @@ int main(int argc, char **argv)
if (!verbose) {
/* detach client and server */
- if (daemon(0, 0) == -1) {
+ if (qemu_daemon(0, 0) == -1) {
err(EXIT_FAILURE, "Failed to daemonize");
}
}
--
1.7.5.3
- [Qemu-devel] [PULL 00/10] Fixes for Cocoa backend, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 01/10] cocoa: do not create a spurious window for -version, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 02/10] Fix compilation warning due to incorrectly specified type, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 03/10] Cocoa: avoid displaying window when command-line contains '-h' or '-help', Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 04/10] Remove warning in printf due to type mismatch, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 05/10] configure: Fix check for fdatasync(), Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 06/10] Fix libfdt warnings on Darwin, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 07/10] cocoa: Provide central qemu_main() prototype, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 08/10] cocoa: Revert dependency on VNC, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 09/10] cocoa: Avoid warning related to multiple handleEvent: definitions, Andreas Färber, 2011/06/13
- [Qemu-devel] [PATCH 10/10] Darwin: Fix compilation warning regarding the deprecated daemon() function,
Andreas Färber <=
- Re: [Qemu-devel] [PATCH 08/10] cocoa: Revert dependency on VNC, Jes Sorensen, 2011/06/27
Re: [Qemu-devel] [PULL 00/10] Fixes for Cocoa backend, Blue Swirl, 2011/06/15