On 12/06/10 16:25, Kevin Wolf wrote: >> } >> n = get_option_parameter(param, BLOCK_OPT_SIZE)->value.n; >> free_option_parameters(param); >> >> bs = bdrv_new_open(filename, fmt, BDRV_O_FLAGS | BDRV_O_RDWR); >> if (!bs) { >> - return 1; >> + ret = -1; >> + goto out; >> } > > Same here. > > Heh, wanted to try it out to be sure, but the compiler notices that, so > it doesn't even build:
Grrrrr I am an idiot! Sorry about the noise, I was sure I had tested that last change. Fix coming up in a few minutes. Cheers, Jes