Description of problem:

struct addrinfo *
host_pton(const char *paddr)
{
        struct addrinfo *ai = NULL;
        struct addrinfo hint = {
                /* don't return duplicates */
                .ai_protocol    = (int)IPPROTO_UDP,
                .ai_flags       = AI_NUMERICHOST,
                .ai_family      = AF_UNSPEC,
        };
        struct sockaddr_in sin;
        int error, inet4;

        /*
         * Although getaddrinfo(3) is easier to use and supports
         * IPv6, it recognizes incomplete addresses like "10.4"
         * as valid AF_INET addresses.  It also accepts presentation
         * addresses that end with a blank.
         *
         * inet_pton(3) is much stricter.  Use it to be certain we
         * have a real AF_INET presentation address, before invoking
         * getaddrinfo(3) to generate the full addrinfo list.
         */
        inet4 = 1;
        if (inet_pton(AF_INET, paddr, &sin.sin_addr) == 0)
                inet4 = 0;

        error = getaddrinfo(paddr, NULL, &hint, &ai);
        switch (error) {
        case 0:
                if (!inet4 && ai->ai_addr->sa_family == AF_INET) {
                        freeaddrinfo(ai);
                        break;
                }
                return ai;
        case EAI_NONAME:
                if (paddr == NULL)  //this is too late,
                ^^^^ if paddr is NULL, the inet_pton() will segment fault.
                     should move to before calling inet_pton()

                        xlog(D_GENERAL, "%s: passed a NULL presentation address",
                                __func__);
                break;

Version-Release number of selected component (if applicable):
nfs-utils-1.3.0

How reproducible:
1

Steps to Reproduce:
1. code review
2.
3.

Actual results:


Expected results:


Additional info: