NotificationsYou must be signed in to change notification settings
Fork4.9k
Star17.7k

Commit59d70a5

committed

Fix one-byte buffer overrun in PQprintTuples().

This bug goes back to the original Postgres95 sources. Its significanceto modern PG versions is marginal, since we have not used PQprintTuples()internally in a very long time, and it doesn't seem to have ever beendocumented either. Still, it *is* exposed to client apps, so somebodyout there might possibly be using it.Xi Wang

1 parent237dc2d commit59d70a5Copy full SHA for 59d70a5

File tree

1 file changed

-4

lines changed

src/interfaces/libpq
- fe-print.c

1 file changed

-4

lines changed

`‎src/interfaces/libpq/fe-print.c`

Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,6 @@ PQprintTuples(const PGresult *res,`
`686`	`686`	`inti,`
`687`	`687`	`j;`
`688`	`688`	`charformatString[80];`
`689`		`-`
`690`	`689`	`char*tborder=NULL;`
`691`	`690`
`692`	`691`	`nFields=PQnfields(res);`
`@@ -705,15 +704,15 @@ PQprintTuples(const PGresult *res,`
`705`	`704`	`intwidth;`
`706`	`705`
`707`	`706`	`width=nFields*14;`
`708`		`-tborder=malloc(width+1);`
	`707`	`+tborder=(char*)malloc(width+1);`
`709`	`708`	`if (!tborder)`
`710`	`709`	`{`
`711`	`710`	`fprintf(stderr,libpq_gettext("out of memory\n"));`
`712`	`711`	`exit(1);`
`713`	`712`	`}`
`714`		`-for (i=0;i <=width;i++)`
	`713`	`+for (i=0;i<width;i++)`
`715`	`714`	`tborder[i]='-';`
`716`		`-tborder[i]='\0';`
	`715`	`+tborder[width]='\0';`
`717`	`716`	`fprintf(fout,"%s\n",tborder);`
`718`	`717`	`}`
`719`	`718`

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!

Commit59d70a5

File tree

1 file changed

1 file changed

`‎src/interfaces/libpq/fe-print.c`

0 commit comments