aboutsummaryrefslogtreecommitdiff
path: root/util/error.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2015-06-19 20:44:54 +0200
committerMarkus Armbruster <armbru@redhat.com>2015-09-10 13:48:06 +0200
commite7cf59e84767e30b507b6bd7c1347072ec12b636 (patch)
tree140bae85557c3e5e13a9fb61ac3734d5f2e6512b /util/error.c
parenta9499ddd82a99c66cc72a08e72427c423acfea1c (diff)
qga: Clean up unnecessarily dirty casts
qga_vss_fsfreeze() casts error_set_win32() from void (*)(Error **, int, ErrorClass, const char *, ...) to void (*)(void **, int, int, const char *, ...) The result is later called. Since the two types are not compatible, the call is undefined behavior. It works in practice anyway. However, there's no real need for trickery here. Clean it up as follows: * Declare struct Error, and fix the first parameter. * Switch to error_setg_win32(). This gets rid of the troublesome ErrorClass parameter. Requires converting error_setg_win32() from macro to function, but that's trivially easy, because this is the only user of error_set_win32(). Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'util/error.c')
-rw-r--r--util/error.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/util/error.c b/util/error.c
index 8f12f67012..9620f2a1f6 100644
--- a/util/error.c
+++ b/util/error.c
@@ -96,8 +96,7 @@ void error_setg_file_open(Error **errp, int os_errno, const char *filename)
#ifdef _WIN32
-void error_set_win32(Error **errp, int win32_err, ErrorClass err_class,
- const char *fmt, ...)
+void error_setg_win32(Error **errp, int win32_err, const char *fmt, ...)
{
va_list ap;
char *msg1, *msg2;
@@ -107,7 +106,7 @@ void error_set_win32(Error **errp, int win32_err, ErrorClass err_class,
}
va_start(ap, fmt);
- error_setv(errp, err_class, fmt, ap);
+ error_setv(errp, ERROR_CLASS_GENERIC_ERROR, fmt, ap);
va_end(ap);
if (win32_err != 0) {