mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-08-27 19:11:16 +00:00
virqemu: Don't strip the requested key from nested objects
Skipping of a specific key is needed only for the top level object to specially handle the object type. We must not pass it to any recursed printing of nested objects as skipping keys there might be surprising and also is unhandlable later when formatting the commandline. Until now this did not pose a problem but was discovered when adding a new netdev backend which has a nested config object which also has the 'type' key which was being skipped. Modern usage will prefer JSON directly but fix the commandline generator to prevent surprises. Signed-off-by: Peter Krempa <pkrempa@redhat.com> Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
parent
9193bac260
commit
3bac06b278
@ -163,8 +163,7 @@ virQEMUBuildCommandLineJSONIterate(const char *key,
|
||||
if (data->prefix)
|
||||
key = tmpkey = g_strdup_printf("%s.%s", data->prefix, key);
|
||||
|
||||
return virQEMUBuildCommandLineJSONRecurse(key, value, data->buf,
|
||||
data->skipKey,
|
||||
return virQEMUBuildCommandLineJSONRecurse(key, value, data->buf, NULL,
|
||||
data->arrayFunc, false);
|
||||
}
|
||||
|
||||
@ -222,14 +221,14 @@ virQEMUBuildCommandLineJSONRecurse(const char *key,
|
||||
return -1;
|
||||
}
|
||||
|
||||
if (arrayFunc(key, value, buf, skipKey) < 0) {
|
||||
if (arrayFunc(key, value, buf, NULL) < 0) {
|
||||
/* fallback, treat the array as a non-bitmap, adding the key
|
||||
* for each member */
|
||||
for (i = 0; i < virJSONValueArraySize(value); i++) {
|
||||
elem = virJSONValueArrayGet((virJSONValue *)value, i);
|
||||
|
||||
/* recurse to avoid duplicating code */
|
||||
if (virQEMUBuildCommandLineJSONRecurse(key, elem, buf, skipKey,
|
||||
if (virQEMUBuildCommandLineJSONRecurse(key, elem, buf, NULL,
|
||||
arrayFunc, true) < 0)
|
||||
return -1;
|
||||
}
|
||||
@ -257,7 +256,8 @@ virQEMUBuildCommandLineJSONRecurse(const char *key,
|
||||
* virQEMUBuildCommandLineJSON:
|
||||
* @value: json object containing the value
|
||||
* @buf: otuput buffer
|
||||
* @skipKey: name of key that will be handled separately by caller
|
||||
* @skipKey: name of key inside the top level object that will be handled
|
||||
* separately by caller
|
||||
* @arrayFunc: array formatter function to allow for different syntax
|
||||
*
|
||||
* Formats JSON value object into command line parameters suitable for use with
|
||||
|
Loading…
Reference in New Issue
Block a user