Bad error handling, memory leaks, and lack of blockdev-add support. v2: * Straightforward rebase * PATCH 01: Superfluous conditional dropped, labels renamed, commit message updated accordingly [Eric] * PATCH 02: More pleasant use of @ret, error message capitalized, line break tidied up [Kevin, Eric] * PATCH 05: Whitespace tidied up [Kevin] * PATCH 06: Missing assignment to ret fixed up [Eric] * PATCH 07: Missing assignment to ret fixed up [Kevin] * PATCH 12: Use -1 instead of 0 for unused argument [Niels] * PATCH 14: Better explain compatibility impact in commit message, whitespace tidied up [Eric]
Markus Armbruster (15): sheepdog: Defuse time bomb in sd_open() error handling sheepdog: Fix error handling in sd_snapshot_delete() sheepdog: Fix error handling sd_create() sheepdog: Mark sd_snapshot_delete() lossage FIXME sheepdog: Fix snapshot ID parsing in _open(), _create, _goto() sheepdog: Don't truncate long VDI name in _open(), _create() sheepdog: Report errors in pseudo-filename more usefully sheepdog: Use SocketAddress and socket_connect() sheepdog: Implement bdrv_parse_filename() gluster: Drop assumptions on SocketTransport names gluster: Don't duplicate qapi-util.c's qapi_enum_parse() gluster: Plug memory leaks in qemu_gluster_parse_json() qapi-schema: Rename GlusterServer to SocketAddressFlat qapi-schema: Rename SocketAddressFlat's variant tcp to inet sheepdog: Support blockdev-add block/gluster.c | 127 +++++++-------- block/sheepdog.c | 453 +++++++++++++++++++++++++++++++++++++-------------- qapi-schema.json | 38 +++++ qapi/block-core.json | 73 +++------ 4 files changed, 451 insertions(+), 240 deletions(-) -- 2.7.4