On 06/26/2018 05:40 PM, Paolo Bonzini wrote:
> Currently --help shows "(default '(null)')" for the -k/--socket-path
> option. Fix it by getting the default path in /var/run.
>
> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> ---
> scsi/qemu-pr-helper.c | 15 ++++++++-------
> 1 file changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/scsi/qemu-pr-helper.c b/scsi/qemu-pr-helper.c
> index d0f83176e1..4057cf355c 100644
> --- a/scsi/qemu-pr-helper.c
> +++ b/scsi/qemu-pr-helper.c
> @@ -74,8 +74,16 @@ static int uid = -1;
> static int gid = -1;
> #endif
>
> +static void compute_default_paths(void)
> +{
> + if (!socket_path) {
> + socket_path =
> qemu_get_local_state_pathname("run/qemu-pr-helper.sock");
> + }
> +}
> +
Pre-existing, but pidfile path is computed elsewhere. I'll post a patch
that cleans things up. Not a show stopper for this one though.
Reviewed-by: Michal Privoznik <mpriv...@redhat.com>
Michal
