NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit1deb6e7

committed

Fix PQsetdbLogin() backward compatibility problem.

If pghost == "" and pgport == "" then PQsetdbLogin() fails with aerror message: Is the postmaster running locally and accepting connections on Unix socket '/tmp/.s.PGSQL.0'?I see many applications such as PHP fails due to this behavior.Now if pgport == "", then it is assumed to be a DEF_PGPORT_STR. Thisis the same behavior as the version prior 7.1.

1 parent317215f commit1deb6e7Copy full SHA for 1deb6e7

File tree

1 file changed

-2

lines changed

src/interfaces/libpq
- fe-connect.c

1 file changed

-2

lines changed

`‎src/interfaces/libpq/fe-connect.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-connect.c,v 1.155 2000/12/18 17:33:41 tgl Exp $`
	`11`	`+ * $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-connect.c,v 1.156 2000/12/22 07:59:32 ishii Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -438,7 +438,7 @@ PQsetdbLogin(const char pghost, const char pgport, const char *pgoptions,`
`438`	`438`	`elseif ((tmp=getenv("PGHOST"))!=NULL)`
`439`	`439`	`conn->pghost=strdup(tmp);`
`440`	`440`
`441`		`-if (pgport==NULL)`
	`441`	`+if (pgport==NULL\|\|pgport[0]=='\0')`
`442`	`442`	`{`
`443`	`443`	`if ((tmp=getenv("PGPORT"))==NULL)`
`444`	`444`	`tmp=DEF_PGPORT_STR;`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit1deb6e7

File tree

1 file changed

1 file changed

`‎src/interfaces/libpq/fe-connect.c`

0 commit comments