diff --git a/src/conf/storage_conf.c b/src/conf/storage_conf.c index 0f515c7cbb..fa3454750c 100644 --- a/src/conf/storage_conf.c +++ b/src/conf/storage_conf.c @@ -1643,6 +1643,35 @@ virStorageVolTargetDefFormat(virStorageVolOptionsPtr options, } +static void +virStorageVolDefFormatSourceExtents(virBufferPtr buf, + virStorageVolDefPtr def) +{ + size_t i; + const char *thispath = NULL; + + for (i = 0; i < def->source.nextent; i++) { + if (thispath == NULL || + STRNEQ(thispath, def->source.extents[i].path)) { + if (thispath != NULL) + virBufferAddLit(buf, "\n"); + + virBufferEscapeString(buf, "\n", + def->source.extents[i].path); + } + + virBufferAdjustIndent(buf, 2); + virBufferAsprintf(buf, "\n", + def->source.extents[i].start, + def->source.extents[i].end); + virBufferAdjustIndent(buf, -2); + thispath = def->source.extents[i].path; + } + if (thispath != NULL) + virBufferAddLit(buf, "\n"); +} + + char * virStorageVolDefFormat(virStoragePoolDefPtr pool, virStorageVolDefPtr def) @@ -1663,29 +1692,8 @@ virStorageVolDefFormat(virStoragePoolDefPtr pool, virBufferAddLit(&buf, "\n"); virBufferAdjustIndent(&buf, 2); - if (def->source.nextent) { - size_t i; - const char *thispath = NULL; - for (i = 0; i < def->source.nextent; i++) { - if (thispath == NULL || - STRNEQ(thispath, def->source.extents[i].path)) { - if (thispath != NULL) - virBufferAddLit(&buf, "\n"); - - virBufferEscapeString(&buf, "\n", - def->source.extents[i].path); - } - - virBufferAdjustIndent(&buf, 2); - virBufferAsprintf(&buf, "\n", - def->source.extents[i].start, - def->source.extents[i].end); - virBufferAdjustIndent(&buf, -2); - thispath = def->source.extents[i].path; - } - if (thispath != NULL) - virBufferAddLit(&buf, "\n"); - } + if (def->source.nextent) + virStorageVolDefFormatSourceExtents(&buf, def); virBufferAdjustIndent(&buf, -2); virBufferAddLit(&buf, "\n");