diff options
author | Michael Roth <mdroth@linux.vnet.ibm.com> | 2016-02-24 18:14:52 -0600 |
---|---|---|
committer | Michael Roth <mdroth@linux.vnet.ibm.com> | 2016-02-25 10:54:32 -0600 |
commit | e55eb806dbb97f53794b0c2f86983d34f6696845 (patch) | |
tree | bc600d543a5e21954572566c47656300824d142d /qga/vss-win32.c | |
parent | 8021de10131868a8857e64b91cf0a868b76a61d8 (diff) | |
download | qemu-e55eb806dbb97f53794b0c2f86983d34f6696845.zip qemu-e55eb806dbb97f53794b0c2f86983d34f6696845.tar.gz qemu-e55eb806dbb97f53794b0c2f86983d34f6696845.tar.bz2 |
qga: fix w32 breakage due to missing osdep.h includes
requester.h relied on qemu/compiler.h definitions to
handle GCC_FMT_ATTR() stub, but this include was removed as part
of scripted clean-ups via 30456d5:
all: Clean up includes
under the assumption that all C files would have included it via
qemu/osdep.h at that point. requester.cpp was likely missed
due to C++ files requiring manual/special handling as well as
VSS build options needing to be enabled to trigger build failures.
Fix this by including qemu/osdep.h. That in turn pulls in a
macro from qapi/error.h that conflicts with a struct field name
in requester.h, so fix that as well by renaming the field.
While we're at it, fix up provider.cpp/install.cpp to include
osdep.h as well.
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'qga/vss-win32.c')
-rw-r--r-- | qga/vss-win32.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qga/vss-win32.c b/qga/vss-win32.c index 5182e3b..9a0e463 100644 --- a/qga/vss-win32.c +++ b/qga/vss-win32.c @@ -150,7 +150,7 @@ void qga_vss_fsfreeze(int *nr_volume, Error **errp, bool freeze) const char *func_name = freeze ? "requester_freeze" : "requester_thaw"; QGAVSSRequesterFunc func; ErrorSet errset = { - .error_setg_win32 = error_setg_win32_internal, + .error_setg_win32_wrapper = error_setg_win32_internal, .errp = errp, }; |