[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 04/27] libvhost-user: drop dependency on glib
From: |
Marc-André Lureau |
Subject: |
[Qemu-devel] [PATCH v2 04/27] libvhost-user: drop dependency on glib |
Date: |
Tue, 19 Sep 2017 18:52:03 +0200 |
libvhost-user is meant to be free of glib dependency. Make sure it is
by droping qemu/osdep.h (which included glib.h)
This fixes a bad malloc()/g_free() pair.
Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
---
contrib/libvhost-user/libvhost-user.c | 29 ++++++++++++++++++++++++++---
1 file changed, 26 insertions(+), 3 deletions(-)
diff --git a/contrib/libvhost-user/libvhost-user.c
b/contrib/libvhost-user/libvhost-user.c
index d27d6303db..50657c5afb 100644
--- a/contrib/libvhost-user/libvhost-user.c
+++ b/contrib/libvhost-user/libvhost-user.c
@@ -13,14 +13,35 @@
* later. See the COPYING file in the top-level directory.
*/
-#include <qemu/osdep.h>
+/* this code avoids GLib dependency */
+#include <stdlib.h>
+#include <stdio.h>
+#include <unistd.h>
+#include <stdarg.h>
+#include <errno.h>
+#include <string.h>
+#include <assert.h>
+#include <inttypes.h>
+#include <sys/types.h>
+#include <sys/socket.h>
#include <sys/eventfd.h>
+#include <sys/mman.h>
#include <linux/vhost.h>
#include "qemu/atomic.h"
+#include "qemu/compiler.h"
#include "libvhost-user.h"
+/* usually provided by GLib */
+#ifndef MIN
+#define MIN(x, y) ({ \
+ typeof(x) _min1 = (x); \
+ typeof(y) _min2 = (y); \
+ (void) (&_min1 == &_min2); \
+ _min1 < _min2 ? _min1 : _min2; })
+#endif
+
#define VHOST_USER_HDR_SIZE offsetof(VhostUserMsg, payload.u64)
/* The version of the protocol we support */
@@ -81,7 +102,9 @@ vu_panic(VuDev *dev, const char *msg, ...)
va_list ap;
va_start(ap, msg);
- buf = g_strdup_vprintf(msg, ap);
+ if (vasprintf(&buf, msg, ap) < 0) {
+ buf = NULL;
+ }
va_end(ap);
dev->broken = true;
@@ -853,7 +876,7 @@ vu_dispatch(VuDev *dev)
success = true;
end:
- g_free(vmsg.data);
+ free(vmsg.data);
return success;
}
--
2.14.1.146.gd35faa819
- [Qemu-devel] [PATCH v2 00/27] vhost-user-scsi: code clean-up, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 01/27] glib-compat: move G_SOURCE_CONTINUE/REMOVE there, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 03/27] build-sys: make vhost-user-scsi depend on libvhost-user.a, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 02/27] build-sys: fix libvhost-user.a build, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 04/27] libvhost-user: drop dependency on glib,
Marc-André Lureau <=
- [Qemu-devel] [PATCH v2 05/27] libvhost-user: improve vu_queue_pop() doc, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 06/27] vhost-user-scsi: use g_strdup(), Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 07/27] vhost-user-scsi: connect unix socket before allocating, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 08/27] vhost-user-scsi: code style fixes, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 09/27] vhost-user-scsi: use glib allocation, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 10/27] vhost-user-scsi: glib calls that allocate don't return NULL, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 11/27] vhost-user-scsi: also free the gtree, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 12/27] vhost-user-scsi: remove vdev_scsi_find_by_vu(), Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 13/27] vhost-user-scsi: simplify unix path cleanup, Marc-André Lureau, 2017/09/19
- [Qemu-devel] [PATCH v2 14/27] vhost-user-scsi: use NULL pointer, Marc-André Lureau, 2017/09/19