Peter Maydell <[email protected]> writes: > On 2 March 2017 at 21:43, Markus Armbruster <[email protected]> wrote: >> Bad error handling, memory leaks, and lack of blockdev-add support. >> >> 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 | 436 >> +++++++++++++++++++++++++++++++++++++-------------- >> qapi-schema.json | 38 +++++ >> qapi/block-core.json | 73 +++------ >> 4 files changed, 443 insertions(+), 231 deletions(-) > > Did you find these by looking at our Coverity results, or > cross-reference them against Coverity? (Coverity definitely > found the gluster leaks, for instance.)
Neither. I merely hacked myself a path through an unfamliar jungle :)
