rpms/kvm/devel kvm-62-block-rw-range-check.patch, NONE, 1.1 .cvsignore, 1.26, 1.27 kvm.spec, 1.43, 1.44 kvm-61-block-rw-range-check.patch, 1.1, NONE
Daniel P. Berrange (berrange)
fedora-extras-commits at redhat.com
Thu Feb 28 00:37:47 UTC 2008
- Previous message (by thread): rpms/python-turboflot/devel .cvsignore, 1.5, 1.6 python-turboflot.spec, 1.4, 1.5 sources, 1.5, 1.6
- Next message (by thread): rpms/kvm/F-8 kvm-60-block-rw-range-check.patch, 1.1, 1.2 kvm.spec, 1.30, 1.31
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
Author: berrange
Update of /cvs/extras/rpms/kvm/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv4592
Modified Files:
.cvsignore kvm.spec
Added Files:
kvm-62-block-rw-range-check.patch
Removed Files:
kvm-61-block-rw-range-check.patch
Log Message:
Fix block device checks for extendable disk formats (rhbz #434978)
kvm-62-block-rw-range-check.patch:
--- NEW FILE kvm-62-block-rw-range-check.patch ---
diff -rup kvm-62.orig/qemu/block.c kvm-62.new/qemu/block.c
--- kvm-62.orig/qemu/block.c 2008-02-27 19:30:21.000000000 -0500
+++ kvm-62.new/qemu/block.c 2008-02-27 19:30:46.000000000 -0500
@@ -124,6 +124,60 @@ void path_combine(char *dest, int dest_s
}
}
+static int bdrv_rd_badreq_sectors(BlockDriverState *bs,
+ int64_t sector_num, int nb_sectors)
+{
+ return
+ nb_sectors < 0 ||
+ sector_num < 0 ||
+ nb_sectors > bs->total_sectors ||
+ sector_num > bs->total_sectors - nb_sectors;
+}
+
+static int bdrv_rd_badreq_bytes(BlockDriverState *bs,
+ int64_t offset, int count)
+{
+ int64_t size = bs->total_sectors << SECTOR_BITS;
+ return
+ count < 0 ||
+ size < 0 ||
+ count > size ||
+ offset > size - count;
+}
+
+static int bdrv_wr_badreq_sectors(BlockDriverState *bs,
+ int64_t sector_num, int nb_sectors)
+{
+ if (sector_num < 0 ||
+ nb_sectors < 0)
+ return 1;
+
+ if (sector_num > bs->total_sectors - nb_sectors) {
+ if (bs->autogrow)
+ bs->total_sectors = sector_num + nb_sectors;
+ else
+ return 1;
+ }
+ return 0;
+}
+
+static int bdrv_wr_badreq_bytes(BlockDriverState *bs,
+ int64_t offset, int count)
+{
+ int64_t size = bs->total_sectors << SECTOR_BITS;
+ if (count < 0 ||
+ offset < 0)
+ return 1;
+
+ if (offset > size - count) {
+ if (bs->autogrow)
+ bs->total_sectors = (offset + count + SECTOR_SIZE - 1) >> SECTOR_BITS;
+ else
+ return 1;
+ }
+ return 0;
+}
+
static void bdrv_register(BlockDriver *bdrv)
{
@@ -332,6 +386,10 @@ int bdrv_open2(BlockDriverState *bs, con
bs->read_only = 0;
bs->is_temporary = 0;
bs->encrypted = 0;
+ bs->autogrow = 0;
+
+ if (flags & BDRV_O_AUTOGROW)
+ bs->autogrow = 1;
if (flags & BDRV_O_SNAPSHOT) {
BlockDriverState *bs1;
@@ -376,6 +434,7 @@ int bdrv_open2(BlockDriverState *bs, con
}
bs->drv = drv;
bs->opaque = qemu_mallocz(drv->instance_size);
+ bs->total_sectors = 0; /* driver will set if it does not do getlength */
if (bs->opaque == NULL && drv->instance_size > 0)
return -1;
/* Note: for compatibility, we open disk image files as RDWR, and
@@ -441,6 +500,7 @@ void bdrv_close(BlockDriverState *bs)
bs->drv = NULL;
/* call the change callback */
+ bs->total_sectors = 0;
bs->media_changed = 1;
if (bs->change_cb)
bs->change_cb(bs->change_opaque);
@@ -506,6 +566,8 @@ int bdrv_read(BlockDriverState *bs, int6
if (!drv)
return -ENOMEDIUM;
+ if (bdrv_rd_badreq_sectors(bs, sector_num, nb_sectors))
+ return -EDOM;
if (sector_num == 0 && bs->boot_sector_enabled && nb_sectors > 0) {
memcpy(buf, bs->boot_sector_data, 512);
sector_num++;
@@ -546,6 +608,8 @@ int bdrv_write(BlockDriverState *bs, int
return -ENOMEDIUM;
if (bs->read_only)
return -EACCES;
+ if (bdrv_wr_badreq_sectors(bs, sector_num, nb_sectors))
+ return -EDOM;
if (sector_num == 0 && bs->boot_sector_enabled && nb_sectors > 0) {
memcpy(bs->boot_sector_data, buf, 512);
}
@@ -671,6 +735,8 @@ int bdrv_pread(BlockDriverState *bs, int
return -ENOMEDIUM;
if (!drv->bdrv_pread)
return bdrv_pread_em(bs, offset, buf1, count1);
+ if (bdrv_rd_badreq_bytes(bs, offset, count1))
+ return -EDOM;
return drv->bdrv_pread(bs, offset, buf1, count1);
}
@@ -686,6 +752,8 @@ int bdrv_pwrite(BlockDriverState *bs, in
return -ENOMEDIUM;
if (!drv->bdrv_pwrite)
return bdrv_pwrite_em(bs, offset, buf1, count1);
+ if (bdrv_wr_badreq_bytes(bs, offset, count1))
+ return -EDOM;
return drv->bdrv_pwrite(bs, offset, buf1, count1);
}
@@ -1091,6 +1159,8 @@ int bdrv_write_compressed(BlockDriverSta
return -ENOMEDIUM;
if (!drv->bdrv_write_compressed)
return -ENOTSUP;
+ if (bdrv_wr_badreq_sectors(bs, sector_num, nb_sectors))
+ return -EDOM;
return drv->bdrv_write_compressed(bs, sector_num, buf, nb_sectors);
}
@@ -1237,6 +1307,8 @@ BlockDriverAIOCB *bdrv_aio_read(BlockDri
if (!drv)
return NULL;
+ if (bdrv_rd_badreq_sectors(bs, sector_num, nb_sectors))
+ return NULL;
/* XXX: we assume that nb_sectors == 0 is suppored by the async read */
if (sector_num == 0 && bs->boot_sector_enabled && nb_sectors > 0) {
@@ -1268,6 +1340,8 @@ BlockDriverAIOCB *bdrv_aio_write(BlockDr
return NULL;
if (bs->read_only)
return NULL;
+ if (bdrv_wr_badreq_sectors(bs, sector_num, nb_sectors))
+ return NULL;
if (sector_num == 0 && bs->boot_sector_enabled && nb_sectors > 0) {
memcpy(bs->boot_sector_data, buf, 512);
}
diff -rup kvm-62.orig/qemu/block.h kvm-62.new/qemu/block.h
--- kvm-62.orig/qemu/block.h 2008-02-25 04:30:14.000000000 -0500
+++ kvm-62.new/qemu/block.h 2008-02-27 19:30:46.000000000 -0500
@@ -45,6 +45,7 @@ typedef struct QEMUSnapshotInfo {
it (default for
bdrv_file_open()) */
#define BDRV_O_DIRECT 0x0020
+#define BDRV_O_AUTOGROW 0x0040 /* Allow backing file to extend when writing past end of file */
#ifndef QEMU_IMG
void bdrv_info(void);
diff -rup kvm-62.orig/qemu/block_int.h kvm-62.new/qemu/block_int.h
--- kvm-62.orig/qemu/block_int.h 2008-02-25 04:30:14.000000000 -0500
+++ kvm-62.new/qemu/block_int.h 2008-02-27 19:30:46.000000000 -0500
@@ -97,6 +97,7 @@ struct BlockDriverState {
int locked; /* if true, the media cannot temporarily be ejected */
int encrypted; /* if true, the media is encrypted */
int sg; /* if true, the device is a /dev/sg* */
+ int autogrow; /* if true, the backing store can auto-extend to allocate new extents */
/* event callback when inserting/removing */
void (*change_cb)(void *opaque);
void *change_opaque;
diff -rup kvm-62.orig/qemu/block-qcow2.c kvm-62.new/qemu/block-qcow2.c
--- kvm-62.orig/qemu/block-qcow2.c 2008-02-25 04:30:14.000000000 -0500
+++ kvm-62.new/qemu/block-qcow2.c 2008-02-27 19:30:46.000000000 -0500
@@ -191,7 +191,7 @@ static int qcow_open(BlockDriverState *b
int len, i, shift, ret;
QCowHeader header;
- ret = bdrv_file_open(&s->hd, filename, flags);
+ ret = bdrv_file_open(&s->hd, filename, flags | BDRV_O_AUTOGROW);
if (ret < 0)
return ret;
if (bdrv_pread(s->hd, 0, &header, sizeof(header)) != sizeof(header))
diff -rup kvm-62.orig/qemu/block-qcow.c kvm-62.new/qemu/block-qcow.c
--- kvm-62.orig/qemu/block-qcow.c 2008-02-25 04:30:14.000000000 -0500
+++ kvm-62.new/qemu/block-qcow.c 2008-02-27 19:30:46.000000000 -0500
@@ -95,7 +95,7 @@ static int qcow_open(BlockDriverState *b
int len, i, shift, ret;
QCowHeader header;
- ret = bdrv_file_open(&s->hd, filename, flags);
+ ret = bdrv_file_open(&s->hd, filename, flags | BDRV_O_AUTOGROW);
if (ret < 0)
return ret;
if (bdrv_pread(s->hd, 0, &header, sizeof(header)) != sizeof(header))
diff -rup kvm-62.orig/qemu/block-vmdk.c kvm-62.new/qemu/block-vmdk.c
--- kvm-62.orig/qemu/block-vmdk.c 2008-02-25 04:30:14.000000000 -0500
+++ kvm-62.new/qemu/block-vmdk.c 2008-02-27 19:30:46.000000000 -0500
@@ -377,7 +377,7 @@ static int vmdk_open(BlockDriverState *b
flags = BDRV_O_RDONLY;
fprintf(stderr, "(VMDK) image open: flags=0x%x filename=%s\n", flags, bs->filename);
- ret = bdrv_file_open(&s->hd, filename, flags);
+ ret = bdrv_file_open(&s->hd, filename, flags | BDRV_O_AUTOGROW);
if (ret < 0)
return ret;
if (bdrv_pread(s->hd, 0, &magic, sizeof(magic)) != sizeof(magic))
Index: .cvsignore
===================================================================
RCS file: /cvs/extras/rpms/kvm/devel/.cvsignore,v
retrieving revision 1.26
retrieving revision 1.27
diff -u -r1.26 -r1.27
--- .cvsignore 26 Feb 2008 19:10:17 -0000 1.26
+++ .cvsignore 28 Feb 2008 00:37:13 -0000 1.27
@@ -1 +1,5 @@
+.*log
+*.rpm
+i686
+x86_64
kvm-62.tar.gz
Index: kvm.spec
===================================================================
RCS file: /cvs/extras/rpms/kvm/devel/kvm.spec,v
retrieving revision 1.43
retrieving revision 1.44
diff -u -r1.43 -r1.44
--- kvm.spec 26 Feb 2008 19:10:17 -0000 1.43
+++ kvm.spec 28 Feb 2008 00:37:13 -0000 1.44
@@ -1,14 +1,14 @@
Summary: Kernel-based Virtual Machine
Name: kvm
Version: 62
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+ and LGPLv2+
Group: Development/Tools
URL: http://%{name}.sf.net
Source0: http://download.sourceforge.net/sourceforge/%{name}/%{name}-%{version}.tar.gz
Source1: %{name}.modules
Patch1: %{name}-bootmenu.patch
-Patch2: %{name}-61-block-rw-range-check.patch
+Patch2: %{name}-62-block-rw-range-check.patch
# patches from upstream qemu
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildRequires: SDL-devel
@@ -93,6 +93,9 @@
%{_sysconfdir}/sysconfig/modules/%{name}.modules
%changelog
+* Wed Feb 27 2008 Daniel P. Berrange <berrange at redhat.com> - 62-2.fc9
+- Fix block device checks for extendable disk formats (rhbz #434978)
+
* Tue Feb 26 2008 Jeremy Katz <katzj at redhat.com> - 62-1
- Update to kvm-62
--- kvm-61-block-rw-range-check.patch DELETED ---
- Previous message (by thread): rpms/python-turboflot/devel .cvsignore, 1.5, 1.6 python-turboflot.spec, 1.4, 1.5 sources, 1.5, 1.6
- Next message (by thread): rpms/kvm/F-8 kvm-60-block-rw-range-check.patch, 1.1, 1.2 kvm.spec, 1.30, 1.31
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the fedora-extras-commits
mailing list