]> granicus.if.org Git - postgresql/commitdiff
Add comment explaining possible compiler warning:
authorBruce Momjian <bruce@momjian.us>
Mon, 29 Nov 2004 03:01:54 +0000 (03:01 +0000)
committerBruce Momjian <bruce@momjian.us>
Mon, 29 Nov 2004 03:01:54 +0000 (03:01 +0000)
    /*
     *  Some compilers with throw a warning knowing this test can never be
     *  true because off_t can't exceed the compared maximum.
     */
    if (th->fileLen > MAX_TAR_MEMBER_FILELEN)
        die_horribly(AH, modulename, "archive member too large for tar format\n");

src/bin/pg_dump/pg_backup_tar.c

index 61eeb6d06db4a3e68ecab21aff270eaf9ce85dce..97b5c3576e31859e3e3fb69a7acbb9ce0c290cd5 100644 (file)
@@ -16,7 +16,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_tar.c,v 1.45 2004/10/07 15:21:55 momjian Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_tar.c,v 1.46 2004/11/29 03:01:54 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1019,6 +1019,10 @@ _tarAddFile(ArchiveHandle *AH, TAR_MEMBER *th)
         */
        fseeko(tmp, 0, SEEK_END);
        th->fileLen = ftello(tmp);
+       /*
+        *      Some compilers with throw a warning knowing this test can never be
+        *      true because off_t can't exceed the compared maximum.
+        */
        if (th->fileLen > MAX_TAR_MEMBER_FILELEN)
                die_horribly(AH, modulename, "archive member too large for tar format\n");
        fseeko(tmp, 0, SEEK_SET);