NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitde52198

committed

Fix latent crash in do_text_output_multiline().

do_text_output_multiline() would fail (typically with a null pointerdereference crash) if its input string did not end with a newline. Suchcases do not arise in our current sources; but it certainly could happenin future, or in extension code's usage of the function, so we should fixit. 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 goodidea to back-patch the fix, since it's a very simple/safe patch and it'snot out of the realm of possibility that we might in future back-patchsomething that expects sane behavior from do_text_output_multiline().Per report from Hao Lee.Report: <CAGoxFiFPAGyPAJLcFxTB5cGhTW2yOVBDYeqDugYwV4dEd1L_Ag@mail.gmail.com>

1 parent7ac0342 commitde52198Copy full SHA for de52198

File tree

2 files changed

+10

-11

lines changed

src
- backend/executor
  - execTuples.c
- include/executor
  - executor.h

2 files changed

+10

-11

lines changed

`‎src/backend/executor/execTuples.c`

Lines changed: 9 additions & 10 deletions

Original file line number	Diff line number	Diff line change
`@@ -1283,33 +1283,32 @@ do_tup_output(TupOutputState tstate, Datum values, bool *isnull)`
`1283`	`1283`	`* Should only be used with a single-TEXT-attribute tupdesc.`
`1284`	`1284`	`*/`
`1285`	`1285`	`void`
`1286`		`-do_text_output_multiline(TupOutputStatetstate,chartext)`
	`1286`	`+do_text_output_multiline(TupOutputStatetstate,constchartxt)`
`1287`	`1287`	`{`
`1288`	`1288`	`Datumvalues[1];`
`1289`	`1289`	`boolisnull[1]= {false};`
`1290`	`1290`
`1291`		`-while (*text)`
	`1291`	`+while (*txt)`
`1292`	`1292`	`{`
`1293`		`-char*eol;`
	`1293`	`+constchar*eol;`
`1294`	`1294`	`intlen;`
`1295`	`1295`
`1296`		`-eol=strchr(text,'\n');`
	`1296`	`+eol=strchr(txt,'\n');`
`1297`	`1297`	`if (eol)`
`1298`	`1298`	`{`
`1299`		`-len=eol-text;`
`1300`		`-`
	`1299`	`+len=eol-txt;`
`1301`	`1300`	`eol++;`
`1302`	`1301`	`}`
`1303`	`1302`	`else`
`1304`	`1303`	`{`
`1305`		`-len=strlen(text);`
`1306`		`-eol+=len;`
	`1304`	`+len=strlen(txt);`
	`1305`	`+eol=txt+len;`
`1307`	`1306`	`}`
`1308`	`1307`
`1309`		`-values[0]=PointerGetDatum(cstring_to_text_with_len(text,len));`
	`1308`	`+values[0]=PointerGetDatum(cstring_to_text_with_len(txt,len));`
`1310`	`1309`	`do_tup_output(tstate,values,isnull);`
`1311`	`1310`	`pfree(DatumGetPointer(values[0]));`
`1312`		`-text=eol;`
	`1311`	`+txt=eol;`
`1313`	`1312`	`}`
`1314`	`1313`	`}`
`1315`	`1314`

`‎src/include/executor/executor.h`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -276,7 +276,7 @@ typedef struct TupOutputState`
`276`	`276`	`externTupOutputStatebegin_tup_output_tupdesc(DestReceiverdest,`
`277`	`277`	`TupleDesctupdesc);`
`278`	`278`	`externvoiddo_tup_output(TupOutputStatetstate,Datumvalues,bool*isnull);`
`279`		`-externvoiddo_text_output_multiline(TupOutputStatetstate,chartext);`
	`279`	`+externvoiddo_text_output_multiline(TupOutputStatetstate,constchartxt);`
`280`	`280`	`externvoidend_tup_output(TupOutputState*tstate);`
`281`	`281`
`282`	`282`	`/*`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitde52198

File tree

2 files changed

2 files changed

`‎src/backend/executor/execTuples.c`

`‎src/include/executor/executor.h`

0 commit comments