[Libguestfs] [PATCH nbdkit 0/2] server: Avoid modifying argv by saving keys in a list and freeing on exit.
Richard W.M. Jones
rjones at redhat.com
Tue Feb 18 11:26:43 UTC 2020
On Tue, Feb 18, 2020 at 10:52:19AM +0000, Richard W.M. Jones wrote:
> Unfortunately you cannot restore argv by setting *p = '=' :-(
>
> The reason is we advertize that plugins are allowed to save they
> ‘const char *key’ pointer passed to them in .config, but assigning
> *p = '=' changes the key string from "key" back to "key=value".
>
> Surprisingly only test-eval.sh actually broke, but other plugins are
> undoubtedly affected.
Most plugins only compare key to some string, but the eval plugin
actually saves the key which is why it fails.
I have pushed the revert commit (part 1).
Rich.
> My alternate fix is fairly horrible, but passes all the tests and
> valgrind.
>
> Rich.
>
>
>
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at redhat.com
> https://www.redhat.com/mailman/listinfo/libguestfs
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW
More information about the Libguestfs
mailing list