summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorRichard W.M. Jones <rjones@redhat.com>2010-10-21 23:06:24 +0100
committerRichard W.M. Jones <rjones@redhat.com>2010-10-22 17:45:06 +0100
commit1250d553f1bec4365b73ec96532fd1de0f85ea94 (patch)
tree5db060a1380b778791aaf3e7902cd867cbb7da75 /tools
parent641ccab6c3b17f1c94676eab99e8baa9cddf5a0b (diff)
downloadlibguestfs-1250d553f1bec4365b73ec96532fd1de0f85ea94.tar.gz
libguestfs-1250d553f1bec4365b73ec96532fd1de0f85ea94.tar.xz
libguestfs-1250d553f1bec4365b73ec96532fd1de0f85ea94.zip
df: Specify format of disks (RHBZ#642934,CVE-2010-3851).
The format parameter is taken from libvirt if available, else the user should supply the '--format' parameter (eg. for local disk files).
Diffstat (limited to 'tools')
-rwxr-xr-xtools/virt-df53
1 files changed, 47 insertions, 6 deletions
diff --git a/tools/virt-df b/tools/virt-df
index d9ef8f9a..61fcfb8e 100755
--- a/tools/virt-df
+++ b/tools/virt-df
@@ -112,6 +112,23 @@ read L</NOTE ABOUT CSV FORMAT> below.
=cut
+my $format;
+
+=item B<--format> raw
+
+Specify the format of disk images given on the command line. If this
+is omitted then the format is autodetected from the content of the
+disk image.
+
+If disk images are requested from libvirt, then this program asks
+libvirt for this information. In this case, the value of the format
+parameter is ignored.
+
+If working with untrusted raw-format guest disk images, you should
+ensure the format is always specified.
+
+=cut
+
my $human;
=item B<--human-readable> | B<-h>
@@ -164,6 +181,7 @@ GetOptions ("help|?" => \$help,
"version" => \$version,
"connect|c=s" => \$uri,
"csv" => \$csv,
+ "format=s" => \$format,
"human-readable|human|h" => \$human,
"inodes|i" => \$inodes,
"one-per-guest" => \$one_per_guest,
@@ -229,7 +247,7 @@ if (@ARGV == 0) { # No params, use libvirt.
} elsif (@ARGV == 1) { # One param, could be disk image or domname.
if (-e $ARGV[0]) {
push @domains, { name => basename ($ARGV[0]),
- disks => [ $ARGV[0] ] }
+ disks => [ [ $ARGV[0], $format ] ] }
} else {
my $conn;
@@ -248,8 +266,9 @@ if (@ARGV == 0) { # No params, use libvirt.
disks => \@disks }
}
} else { # >= 2 params, all disk images.
+ my @disks = map { [ $_, $format ] } @ARGV;
push @domains, { name => basename ($ARGV[0]),
- disks => \@ARGV }
+ disks => \@disks }
}
sub get_disks_from_libvirt
@@ -258,12 +277,29 @@ sub get_disks_from_libvirt
my $xml = $dom->get_xml_description ();
my $p = XML::XPath->new (xml => $xml);
- my @disks = $p->findnodes ('//devices/disk/source/@dev');
- push (@disks, $p->findnodes ('//devices/disk/source/@file'));
+ my $nodes = $p->find ('//devices/disk');
+
+ my @disks;
+ my $node;
+ foreach $node ($nodes->get_nodelist) {
+ # The filename can be in dev or file attribute, hence:
+ my $filename = $p->find ('./source/@dev', $node);
+ unless ($filename) {
+ $filename = $p->find ('./source/@file', $node);
+ next unless $filename;
+ }
+ $filename = $filename->to_literal;
+
+ # Get the disk format (may not be set).
+ my $format = $p->find ('./driver/@type', $node);
+ $format = $format->to_literal if $format;
+
+ push @disks, [ $filename, $format ];
+ }
# Code in Sys::Guestfs::Lib dies here if there are no disks at all.
- return map { $_->getData } @disks;
+ return @disks;
}
# Sort the domains by name for display.
@@ -312,7 +348,12 @@ sub multi_df
foreach $d (@_) {
foreach $disk (@{$d->{disks}}) {
- $g->add_drive_ro ($disk);
+ my $filename = $disk->[0];
+ my $format = $disk->[1];
+ my @args = ($filename);
+ push @args, readonly => 1;
+ push @args, format => $format if defined $format;
+ $g->add_drive_opts (@args);
}
}