[libvirt] [PATCH v2 1/6] qemu: command: Rename and move qemuNetworkDriveGetPort

John Ferlan jferlan at redhat.com
Fri Jul 21 11:58:52 UTC 2017


[...]

Before it's too late...

> +
> +/**
> + * virStringParsePort:
> + * @str: port number to parse
> + * @port: pointer to parse port into
> + *
> + * Parses a string representation of a network port and validates it. Returns
> + * 0 on success and -1 on error.
> + */
> +int
> +virStringParsePort(const char *str,
> +                   int *port)
> +{
> +    unsigned int p = 0;
> +
> +    *port = 0;
> +
> +    if (!str)
> +        return 0;
> +
> +    if (virStrToLong_uip(str, NULL, 10, &p) < 0) {
> +        virReportError(VIR_ERR_INVALID_ARG,
> +                       _("failed to parse port number '%s'"), str);
> +        return -1;
> +    }
> +
> +    if (p > 65535) {

Should this be UINT16_MAX ?


> +        virReportError(VIR_ERR_INVALID_ARG,
> +                       _("port '%s' out of range"), str);
> +        return -1;
> +    }
> +
> +    *port = p;
> +
> +    return 0;
> +}




More information about the libvir-list mailing list