> ------- Original Message -------
> From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> To: Dave Page <dpage(at)postgresql(dot)org>
> Sent: 29/09/07, 01:28:09
> Subject: Re: [PATCHES] OpenSSL Applink
> I concur with Magnus that it'll be better if there's not two code paths
> here. It's not entirely clear whether BIO_new_fp() would avoid the
> problematic calls, but it doesn't look like it'd be hard to try.
The last version of the patch I posted uses BIO_new_file() in all cases, and (from memory) BIO_get_fp() in the non-win32 case to get a FILE* to pass to fstat.
I believe the problem is sharing FILE*'s between differing runtime versions used by openssl and the app. Coding it with BIO_new_file makes it less tempting for future changes to start mucking about with FILE*'s on Windows.
Note that I've been away from my *nix boxes so I haven't tested the patch except on Windows yet.
pgsql-patches by date
|Next:||From: Simon Riggs||Date: 2007-09-29 07:59:21|
|Subject: set_ps_display during recovery|
|Previous:||From: Magnus Hagander||Date: 2007-09-29 07:16:05|
|Subject: Re: Add chklocale.c to msvc build|