diff options
author | Eric Blake <eblake@redhat.com> | 2015-11-05 23:35:31 -0700 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2015-11-10 08:08:21 +0100 |
commit | a12a5a1a0132527afe87c079e4aae4aad372bd94 (patch) | |
tree | 5338234400f554e439fc3566364b53fa08f65ab5 /include/qapi | |
parent | 3f66f764ee25f10d3e1144ebc057a949421b7728 (diff) | |
download | qemu-a12a5a1a0132527afe87c079e4aae4aad372bd94.zip qemu-a12a5a1a0132527afe87c079e4aae4aad372bd94.tar.gz qemu-a12a5a1a0132527afe87c079e4aae4aad372bd94.tar.bz2 |
qapi: Simplify error cleanup in test-qmp-*
We have several tests that perform multiple sub-actions that are
expected to fail. Asserting that an error occurred, then clearing
it up to prepare for the next action, turned into enough
boilerplate that it was sometimes forgotten (for example, a number
of tests added to test-qmp-input-visitor.c in d88f5fd leaked err).
Worse, if an error is not reset to NULL, we risk invalidating
later use of that error (passing a non-NULL err into a function
is generally a bad idea). Encapsulate the boilerplate into a
single helper function error_free_or_abort(), and consistently
use it.
The new function is added into error.c for use everywhere,
although it is anticipated that testsuites will be the main
client.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'include/qapi')
-rw-r--r-- | include/qapi/error.h | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/include/qapi/error.h b/include/qapi/error.h index c69dddb..4d42cdc 100644 --- a/include/qapi/error.h +++ b/include/qapi/error.h @@ -30,6 +30,10 @@ * Handle an error without reporting it (just for completeness): * error_free(err); * + * Assert that an expected error occurred, but clean it up without + * reporting it (primarily useful in testsuites): + * error_free_or_abort(&err); + * * Pass an existing error to the caller: * error_propagate(errp, err); * where Error **errp is a parameter, by convention the last one. @@ -190,6 +194,11 @@ Error *error_copy(const Error *err); void error_free(Error *err); /* + * Convenience function to assert that *@errp is set, then silently free it. + */ +void error_free_or_abort(Error **errp); + +/* * Convenience function to error_report() and free @err. */ void error_report_err(Error *); |