"Bossart, Nathan" <bossa...@amazon.com> writes: > - appendPQExpBuffer(&conn->errorMessage, > - libpq_gettext("out of > memory allocating GSSAPI buffer (%d)\n"), > - payloadlen); > + pqReportOOM(conn);
> I see that some context is lost in a few places (e.g., the one above > points to a GSSAPI buffer). Perhaps this extra context could be > useful to identify problematic areas, but it might be unlikely to help > much in these parts of libpq. In any case, the vast majority of > existing callers don't provide any extra context. Yeah, there are half a dozen places that currently print something more specific than "out of memory". I judged that the value of this was not worth the complexity it'd add to support it in this scheme. Different opinions welcome of course. regards, tom lane