From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: Sloppy SSPI error reporting code |
Date: | 2015-04-08 14:28:55 |
Message-ID: | 20150408142855.GB4785@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Apr 3, 2015 at 08:32:08PM -0400, Noah Misch wrote:
> On Thu, Apr 02, 2015 at 07:31:52AM -0400, Bruce Momjian wrote:
> > On Thu, Apr 2, 2015 at 01:44:59AM -0400, Noah Misch wrote:
> > > On Wed, Apr 01, 2015 at 10:49:01PM -0400, Bruce Momjian wrote:
> > > > On Sat, Jan 10, 2015 at 02:53:13PM -0500, Tom Lane wrote:
> > > > > While looking at fe-auth.c I noticed quite a few places that weren't
> > > > > bothering to make error messages localizable (ie, missing libpq_gettext
> > > > > calls), and/or were failing to add a trailing newline as expected in
> > > > > libpq error messages. Perhaps these are intentional but I doubt it.
> > > > > Most of the instances seemed to be SSPI-related.
> > > > >
> > > > > I have no intention of fixing these myself, but whoever committed that
> > > > > code should take a second look.
> > > >
> > > > I looked through that file and only found two cases; patch attached.
> > >
> > > Tom mentioned newline omissions, which you'll find in pg_SSPI_error().
> >
> > Oh, I accidentally saw the backend version of that function, which
> > looked fine. I have attached a patch for that.
>
> That patch looks reasonable.
Patch applied. Thanks.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ Everyone has their own god. +
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-04-08 14:57:43 | Re: Support UPDATE table SET(*)=... |
Previous Message | Kevin Grittner | 2015-04-08 14:07:41 | Re: Row security violation error is misleading |