Richard Jones <[email protected]> writes: > Public bug reported: > > $ qemu-system-x86_64 -drive file=/tmp/foo\" -writeconfig - > # qemu config file > > [drive] > file = "/tmp/foo"" > > For bonus points, try to construct a valid qemu config file that > contains a quoted value. It's pretty clear (from looking at the code > also) that this is not possible. > > Also: > > - maximum value length is hard-coded in the parser at 1023 characters > (for no apparent reason) > > - the format is undocumented
Perhaps for the better, as -readconfig and -writeconfig are seriously limited in serveral ways. The format is Windows INI, which is as poorly specified as it's widespread. I intend to work on more capable configuration file support, but it'll take some time. > - don't use sscanf for parsing! > > ** Affects: qemu > Importance: Undecided > Status: New > > ** Description changed: > > $ qemu-system-x86_64 -drive file=/tmp/foo\" -writeconfig - > # qemu config file > > [drive] > - file = "/tmp/foo"" > + file = "/tmp/foo"" > > - For bonus points, try to construct a value qemu config file that > + For bonus points, try to construct a valid qemu config file that > contains a quoted value. It's pretty clear (from looking at the code > also) that this is not possible. > > Also: > > - maximum value length is hard-coded in the parser at 1023 characters > (for no apparent reason) > > - the format is undocumented > > - don't use sscanf for parsing!
