Avoid core dump after getpwuid_r failure.

Looking up a nonexistent user ID would lead to a null pointer
dereference.  That's unlikely to happen here, but perhaps
not impossible.

Thinko in commit 4d5111b3f, noticed by Coverity.
pull/177/head
Tom Lane 1 year ago
parent d8df7ac5c0
commit 2e62fa62d6
  1. 5
      src/interfaces/libpq/fe-auth.c

@ -1205,7 +1205,7 @@ pg_fe_getusername(uid_t user_id, PQExpBuffer errorMessage)
DWORD namesize = sizeof(username); DWORD namesize = sizeof(username);
#else #else
struct passwd pwbuf; struct passwd pwbuf;
struct passwd *pw; struct passwd *pw = NULL;
char buf[1024]; char buf[1024];
int rc; int rc;
#endif #endif
@ -1230,7 +1230,8 @@ pg_fe_getusername(uid_t user_id, PQExpBuffer errorMessage)
if (errorMessage) if (errorMessage)
libpq_append_error(errorMessage, "local user with ID %ld does not exist", (long) user_id); libpq_append_error(errorMessage, "local user with ID %ld does not exist", (long) user_id);
} }
name = pw->pw_name; else
name = pw->pw_name;
#endif #endif
if (name) if (name)

Loading…
Cancel
Save