Date: Tue, 28 Oct 2014 21:01:20 -0400 From: Conrad Meyer <cse.cem@gmail.com> To: Michal Privoznik <mprivozn@redhat.com> Cc: libvir-list@redhat.com, "freebsd-virtualization@freebsd.org" <freebsd-virtualization@freebsd.org> Subject: Re: [libvirt] [PATCH 1/2] bhyve: Support /domain/bootloader configuration for non-FreeBSD guests. Message-ID: <CAG6CVpWSGcHR2jU7zCsm8OP584dR2Pt7WHOV5KyQUy9Lpq%2B8tg@mail.gmail.com> In-Reply-To: <54503292.9040907@redhat.com> References: <1414094284-29055-1-git-send-email-cse.cem@gmail.com> <1414094284-29055-2-git-send-email-cse.cem@gmail.com> <54503292.9040907@redhat.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> I don't think it's safe to pass arbitrary arguments from XML. I find this > too critical to ACK the patch, buy maybe further discussion can change my > mind. Michal, You're reviewing a 4-revision old version of this patch, see v5 for the most recent set. However, this bit is still present in v5. What is <bootloader_args> in the XML is for, if not arbitrary arguments? Thanks, Conrad
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpWSGcHR2jU7zCsm8OP584dR2Pt7WHOV5KyQUy9Lpq%2B8tg>