Skip site navigation (1) Skip section navigation (2)

Re: Libpq ssl fix

From: Manfred Spraul <manfred(at)colorfullife(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andreas Pflug <pgadmin(at)pse-consulting(dot)de>,PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Libpq ssl fix
Date: 2004-05-28 18:17:07
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-patches

I'll retest the patch. I didn't have a working ssl test setup, thus I 
stopped when I ran into the same errors as without my patch. Probably 
printfs in initialize_SSL(), but no tests that the code beyond 
initialize_SSL() actually runs - sorry.

Btw, --enable-thread-safety on Linux (RedHat Fedora Core 1) fails in 
configure with

> configure: error:
> *** Thread test program failed.  Your platform is not thread-safe.
> *** Check the file 'config.log'for the exact reason.


Tom Lane wrote:

>Andreas Pflug <pgadmin(at)pse-consulting(dot)de> writes:
>>init_ssl_system  will return 0 on success and -1 on failure, which will 
>>be interpreted just the other way round in initialize_SSL.
>>Patch appended.
>Hmm, that looks backwards to me too, but this would seem to imply that
>Manfred Spraul failed to test his last patch at all.  Manfred, care to
>explain why we shouldn't revert that patch in toto?
>2004-03-23 22:44  momjian
>	* doc/src/sgml/libpq.sgml, src/backend/libpq/md5.c,
>	src/interfaces/libpq/fe-auth.c, src/interfaces/libpq/fe-connect.c,
>	src/interfaces/libpq/fe-secure.c, src/interfaces/libpq/libpq-fe.h,
>	src/interfaces/libpq/libpq-int.h: Add thread locking to SSL and
>	Kerberos connections.
>	I have removed the docs mentioning that SSL and Kerberos are not
>	thread-safe.
>	Manfred Spraul
>			regards, tom lane

In response to


pgsql-patches by date

Next:From: Thomas HallgrenDate: 2004-05-28 18:21:34
Subject: Re: Link errors on win32
Previous:From: Andreas PflugDate: 2004-05-28 17:42:48
Subject: Re: Libpq ssl fix

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group