pgsql: Fix latent crash in do_text_output_multiline(). - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix latent crash in do_text_output_multiline().
Date
Msg-id E1b4uPV-0002Nk-4p@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix latent crash in do_text_output_multiline().

do_text_output_multiline() would fail (typically with a null pointer
dereference crash) if its input string did not end with a newline.  Such
cases do not arise in our current sources; but it certainly could happen
in future, or in extension code's usage of the function, so we should fix
it.  To fix, replace "eol += len" with "eol = text + len".

While at it, make two cosmetic improvements: mark the input string const,
and rename the argument from "text" to "txt" to dodge pgindent strangeness
(since "text" is a typedef name).

Even though this problem is only latent at present, it seems like a good
idea to back-patch the fix, since it's a very simple/safe patch and it's
not out of the realm of possibility that we might in future back-patch
something that expects sane behavior from do_text_output_multiline().

Per report from Hao Lee.

Report: <CAGoxFiFPAGyPAJLcFxTB5cGhTW2yOVBDYeqDugYwV4dEd1L_Ag@mail.gmail.com>

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/7b40b2d90f44c1f4ef1bc130bd94decb15826676

Modified Files
--------------
src/backend/executor/execTuples.c | 19 +++++++++----------
src/include/executor/executor.h   |  2 +-
2 files changed, 10 insertions(+), 11 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix latent crash in do_text_output_multiline().
Next
From: Tom Lane
Date:
Subject: pgsql: Fix latent crash in do_text_output_multiline().