block: mirror - remove code cruft that has no function
authorJeff Cody <jcody@redhat.com>
Tue, 4 Mar 2014 15:35:48 +0000 (10:35 -0500)
committerKevin Wolf <kwolf@redhat.com>
Thu, 6 Mar 2014 10:47:40 +0000 (11:47 +0100)
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>
block/mirror.c

index e683959570610401d2c07c98ee51700d15f59f8a..dd5ee056b4db18c96f5e2a65f27bd7d13f40bd09 100644 (file)
@@ -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;
     }