[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 05/19] Ensure that QEMU exits if drive_add parsing f
From: |
Daniel P. Berrange |
Subject: |
[Qemu-devel] [PATCH 05/19] Ensure that QEMU exits if drive_add parsing fails |
Date: |
Mon, 7 Jun 2010 15:42:18 +0100 |
The drive_add() method returns NULL if it failed to parse the
parameter values for any reason. All callers must check this
and exit if failure occurred. Annotate the method so that the
compiler validates this.
Signed-off-by: Daniel P. Berrange <address@hidden>
---
vl.c | 38 ++++++++++++++++++++++++++------------
1 files changed, 26 insertions(+), 12 deletions(-)
diff --git a/vl.c b/vl.c
index 7121cd0..3d08a44 100644
--- a/vl.c
+++ b/vl.c
@@ -654,6 +654,7 @@ static int bt_parse(const char *opt)
#define MTD_ALIAS "if=mtd"
#define SD_ALIAS "index=0,if=sd"
+QEMU_WARN_UNUSED_RESULT
QemuOpts *drive_add(const char *file, const char *fmt, ...)
{
va_list ap;
@@ -2682,7 +2683,8 @@ int main(int argc, char **argv, char **envp)
if (optind >= argc)
break;
if (argv[optind][0] != '-') {
- hda_opts = drive_add(argv[optind++], HD_ALIAS, 0);
+ if (!(hda_opts = drive_add(argv[optind++], HD_ALIAS, 0)))
+ exit(1);
} else {
const QEMUOption *popt;
@@ -2731,14 +2733,18 @@ int main(int argc, char **argv, char **envp)
",trans=lba" :
translation == BIOS_ATA_TRANSLATION_NONE ?
",trans=none" : "");
- break;
+ if (!hda_opts)
+ exit(1);
+ break;
case QEMU_OPTION_hdb:
case QEMU_OPTION_hdc:
case QEMU_OPTION_hdd:
- drive_add(optarg, HD_ALIAS, popt->index - QEMU_OPTION_hda);
+ if (!drive_add(optarg, HD_ALIAS, popt->index -
QEMU_OPTION_hda))
+ exit(1);
break;
case QEMU_OPTION_drive:
- drive_add(NULL, "%s", optarg);
+ if (!drive_add(NULL, "%s", optarg))
+ exit(1);
break;
case QEMU_OPTION_set:
if (qemu_set_option(optarg) != 0)
@@ -2749,13 +2755,16 @@ int main(int argc, char **argv, char **envp)
exit(1);
break;
case QEMU_OPTION_mtdblock:
- drive_add(optarg, MTD_ALIAS);
+ if (!drive_add(optarg, MTD_ALIAS))
+ exit(1);
break;
case QEMU_OPTION_sd:
- drive_add(optarg, SD_ALIAS);
+ if (!drive_add(optarg, SD_ALIAS))
+ exit(1);
break;
case QEMU_OPTION_pflash:
- drive_add(optarg, PFLASH_ALIAS);
+ if (!drive_add(optarg, PFLASH_ALIAS))
+ exit(1);
break;
case QEMU_OPTION_snapshot:
snapshot = 1;
@@ -2834,7 +2843,8 @@ int main(int argc, char **argv, char **envp)
kernel_cmdline = optarg;
break;
case QEMU_OPTION_cdrom:
- drive_add(optarg, CDROM_ALIAS);
+ if (!drive_add(optarg, CDROM_ALIAS))
+ exit(1);
break;
case QEMU_OPTION_boot:
{
@@ -2887,7 +2897,8 @@ int main(int argc, char **argv, char **envp)
break;
case QEMU_OPTION_fda:
case QEMU_OPTION_fdb:
- drive_add(optarg, FD_ALIAS, popt->index - QEMU_OPTION_fda);
+ if (!drive_add(optarg, FD_ALIAS, popt->index -
QEMU_OPTION_fda))
+ exit(1);
break;
case QEMU_OPTION_no_fd_bootchk:
fd_bootchk = 0;
@@ -3625,17 +3636,20 @@ int main(int argc, char **argv, char **envp)
if (default_cdrom) {
/* we always create the cdrom drive, even if no disk is there */
- drive_add(NULL, CDROM_ALIAS);
+ if (!drive_add(NULL, CDROM_ALIAS))
+ exit(1);
}
if (default_floppy) {
/* we always create at least one floppy */
- drive_add(NULL, FD_ALIAS, 0);
+ if (!drive_add(NULL, FD_ALIAS, 0))
+ exit(1);
}
if (default_sdcard) {
/* we always create one sd slot, even if no card is in it */
- drive_add(NULL, SD_ALIAS);
+ if (!drive_add(NULL, SD_ALIAS))
+ exit(1);
}
/* open the virtual block devices */
--
1.6.6.1
- [Qemu-devel] [PATCH 00/19] RFC: Reporting QEMU binary capabilities, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 01/19] Add support for JSON pretty printing, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 02/19] Add support for compile time assertions, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 03/19] Add enum handlers for easy & efficient string <-> int conversion, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 04/19] Add support for a option parameter as an enum, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 05/19] Ensure that QEMU exits if drive_add parsing fails,
Daniel P. Berrange <=
- [Qemu-devel] [PATCH 08/19] Convert RTC to use enumerations for configuration parameters, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 07/19] Convert netdev client types to use an enumeration, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 09/19] Change 'query-version' to output broken down version string, Daniel P. Berrange, 2010/06/07
- [Qemu-devel] [PATCH 11/19] Add a query-devices command to QMP, Daniel P. Berrange, 2010/06/07