Patchwork [1/3] vvfat: Fix segfault on write to read-only disk

login
register
mail settings
Submitter Kevin Wolf
Date Sept. 10, 2010, 10:27 a.m.
Message ID <1284114424-11960-2-git-send-email-mail@kevin-wolf.de>
Download mbox | patch
Permalink /patch/64370/
State New
Headers show

Comments

Kevin Wolf - Sept. 10, 2010, 10:27 a.m.
vvfat tries to set the readonly flag in its open function, but nowadays
this is overwritted with the readonly=... command line option. Check in
bdrv_write if the vvfat was opened read-only and return an error in this
case.

Without this check, vvfat tries to access the qcow bs, which is NULL
without enabled write support.

Signed-off-by: Kevin Wolf <mail@kevin-wolf.de>
---
 block/vvfat.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

Patch

diff --git a/block/vvfat.c b/block/vvfat.c
index 365332a..5898d66 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -2665,6 +2665,11 @@  static int vvfat_write(BlockDriverState *bs, int64_t sector_num,
 
 DLOG(checkpoint());
 
+    /* Check if we're operating in read-only mode */
+    if (s->qcow == NULL) {
+        return -EACCES;
+    }
+
     vvfat_close_current_file(s);
 
     /*