diff options
| author | Jeff Cody <jcody@redhat.com> | 2014-03-04 10:35:48 -0500 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2014-03-06 11:47:40 +0100 |
| commit | 50c75136beec0180171971c88bb6e3fa3aa66463 (patch) | |
| tree | 179970abec4e49177e768b867ea044493ebe50ed /block | |
| parent | 90ce8a061bdcc485a56142cae68cfbfff270e634 (diff) | |
| download | qemu-50c75136beec0180171971c88bb6e3fa3aa66463.tar.gz | |
block: mirror - remove code cruft that has no function
Originally, this built up the error message with the backing filename,
so that errp was set as follows:
error_set(errp, QERR_OPEN_FILE_FAILED, backing_filename);
However, we now propagate the local_error from the
bdrv_open_backing_file() call instead, making these 2 lines useless
code.
Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/mirror.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/block/mirror.c b/block/mirror.c index e683959570..dd5ee056b4 100644 --- a/block/mirror.c +++ b/block/mirror.c @@ -520,9 +520,6 @@ static void mirror_complete(BlockJob *job, Error **errp) ret = bdrv_open_backing_file(s->target, NULL, &local_err); if (ret < 0) { - char backing_filename[PATH_MAX]; - bdrv_get_full_backing_filename(s->target, backing_filename, - sizeof(backing_filename)); error_propagate(errp, local_err); return; } |