Re: patch for src/backend/main/main.c - Mailing list pgsql-patches

From Tom Lane
Subject Re: patch for src/backend/main/main.c
Date
Msg-id 9963.980297556@sss.pgh.pa.us
Whole thread Raw
In response to patch for src/backend/main/main.c  ("Michael C. Thornburgh" <zenomt@armory.com>)
List pgsql-patches
"Michael C. Thornburgh" <zenomt@armory.com> writes:
> +     len = strlen(pw->pw_name);
> +     pw_name_persist = (char *) malloc(len+1);
> +     if (pw_name_persist == (char *)NULL)
> +     {
> +         fprintf(stderr, "%s: can't malloc for username\n", argv[0]);
> +         exit(1);
> +     }
> +     strncpy(pw_name_persist, pw->pw_name, len+1);

This could be simplified to
    pw_name_persist = strdup(pw->pw_name);
no?

            regards, tom lane

pgsql-patches by date:

Previous
From: "Michael C. Thornburgh"
Date:
Subject: patch for src/backend/main/main.c
Next
From: "Michael C. Thornburgh"
Date:
Subject: Re: patch for src/backend/main/main.c