Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitce7fc4f

Browse files
committed
Fix pg_restore's processing of old-style BLOB COMMENTS data.
Prior to 9.0, pg_dump handled comments on large objects by dumping a bunchof COMMENT commands into a single BLOB COMMENTS archive object. Withsufficiently many such comments, some of the commands would likely getsplit across bufferloads when restoring, causing failures indirect-to-database restores (though no problem would be evident in textoutput). This is the same type of issue we have with table data dumped asINSERT commands, and it can be fixed in the same way, by using a mini SQLlexer to figure out where the command boundaries are. Fortunately, theCOMMENT commands are no more complex to lex than INSERTs, so we can justre-use the existing lexer for INSERTs.Per bug #10611 from Jacek Zalewski. Back-patch to all active branches.
1 parent72adbe5 commitce7fc4f

File tree

3 files changed

+16
-4
lines changed

3 files changed

+16
-4
lines changed

‎src/bin/pg_dump/pg_backup_archiver.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -672,7 +672,13 @@ restore_toc_entry(ArchiveHandle *AH, TocEntry *te,
672672

673673
_selectOutputSchema(AH,"pg_catalog");
674674

675+
/* Send BLOB COMMENTS data to ExecuteSimpleCommands() */
676+
if (strcmp(te->desc,"BLOB COMMENTS")==0)
677+
AH->outputKind=OUTPUT_OTHERDATA;
678+
675679
(*AH->PrintTocDataPtr) (AH,te,ropt);
680+
681+
AH->outputKind=OUTPUT_SQLCMDS;
676682
}
677683
else
678684
{

‎src/bin/pg_dump/pg_backup_db.c

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -425,9 +425,14 @@ ExecuteSqlCommand(ArchiveHandle *AH, const char *qry, const char *desc)
425425
* identifiers, so that we can recognize statement-terminating semicolons.
426426
* We assume that INSERT data will not contain SQL comments, E'' literals,
427427
* or dollar-quoted strings, so this is much simpler than a full SQL lexer.
428+
*
429+
* Note: when restoring from a pre-9.0 dump file, this code is also used to
430+
* process BLOB COMMENTS data, which has the same problem of containing
431+
* multiple SQL commands that might be split across bufferloads. Fortunately,
432+
* that data won't contain anything complicated to lex either.
428433
*/
429434
staticvoid
430-
ExecuteInsertCommands(ArchiveHandle*AH,constchar*buf,size_tbufLen)
435+
ExecuteSimpleCommands(ArchiveHandle*AH,constchar*buf,size_tbufLen)
431436
{
432437
constchar*qry=buf;
433438
constchar*eos=buf+bufLen;
@@ -511,9 +516,10 @@ ExecuteSqlCommandBuf(ArchiveHandle *AH, const char *buf, size_t bufLen)
511516
elseif (AH->outputKind==OUTPUT_OTHERDATA)
512517
{
513518
/*
514-
* Table data expressed as INSERT commands.
519+
* Table data expressed as INSERT commands; or, in old dump files,
520+
* BLOB COMMENTS data (which is expressed as COMMENT ON commands).
515521
*/
516-
ExecuteInsertCommands(AH,buf,bufLen);
522+
ExecuteSimpleCommands(AH,buf,bufLen);
517523
}
518524
else
519525
{

‎src/bin/pg_dump/pg_dump.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1415,7 +1415,7 @@ dumpTableData_copy(Archive *fout, void *dcontext)
14151415
*
14161416
* Caution: when we restore from an archive file direct to database, the
14171417
* INSERT commands emitted by this function have to be parsed by
1418-
* pg_backup_db.c'sExecuteInsertCommands(), which will not handle comments,
1418+
* pg_backup_db.c'sExecuteSimpleCommands(), which will not handle comments,
14191419
* E'' strings, or dollar-quoted strings. So don't emit anything like that.
14201420
*/
14211421
staticint

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp