pg_dump: Replace can't-happen error with assertion
authorPeter Eisentraut <peter@eisentraut.org>
Mon, 27 Apr 2020 12:24:20 +0000 (14:24 +0200)
committerPeter Eisentraut <peter@eisentraut.org>
Mon, 27 Apr 2020 12:24:20 +0000 (14:24 +0200)
src/bin/pg_dump/pg_backup_tar.c

index 775118f2978072fdec58c33bf9f463cefacc0ab4..d5bfa5564615207c544b8e2a0675c166ccb63277 100644 (file)
@@ -515,6 +515,8 @@ _tarReadRaw(ArchiveHandle *AH, void *buf, size_t len, TAR_MEMBER *th, FILE *fh)
        size_t          used = 0;
        size_t          res = 0;
 
+       Assert(th || fh);
+
        avail = AH->lookaheadLen - AH->lookaheadPos;
        if (avail > 0)
        {
@@ -567,8 +569,6 @@ _tarReadRaw(ArchiveHandle *AH, void *buf, size_t len, TAR_MEMBER *th, FILE *fh)
                                        READ_ERROR_EXIT(th->nFH);
                        }
                }
-               else
-                       fatal("internal error -- neither th nor fh specified in _tarReadRaw()");
        }
 
        ctx->tarFHpos += res + used;