Re: Please test peer (socket ident) auth on *BSD
Marko Kreen <markokr@gmail.com>
From: Marko Kreen <markokr@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Greg Stark <gsstark@mit.edu>, Peter Eisentraut <peter_e@gmx.net>, pgsql-hackers@postgresql.org, Bruce Momjian <bruce@momjian.us>
Date: 2011-05-31T22:30:56Z
Lists: pgsql-hackers
On Wed, Jun 1, 2011 at 1:03 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Marko Kreen <markokr@gmail.com> writes: >> My suggestion would be to use getpeereid() everywhere. >> And just have compat getpeereid() implementation on non-BSD >> platforms. This would minimize ifdeffery in core core. > > Hm, maybe. I'd be for this if we had more than two call sites, but > as things stand I'm not sure it's worth the trouble to set up a src/port > module for it. It would remove ~50 lines of low-level code from otherwise high-level function. So even with one call site it would be improvement. If the src/port is trouble, how about putting it as 'static inline' into .h? :) -- marko