Re: patch for src/backend/main/main.c

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Michael C(dot) Thornburgh" <zenomt(at)armory(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: patch for src/backend/main/main.c
Date: 2001-01-24 00:52:36
Message-ID: 9963.980297556@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

"Michael C. Thornburgh" <zenomt(at)armory(dot)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

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-01-24 01:38:38 Re: Re: patch for src/backend/main/main.c
Previous Message Tom Lane 2001-01-23 22:43:07 Re: [PATCHES] binary operators on integers