Prevent core dump when die_horribly() is called with null AH pointer.

Problem reported and fixed by Oliver Elphick.

diff --git a/src/bin/pg_dump/pg_backup_archiver.c b/src/bin/pg_dump/pg_backup_archiver.c
index 2b38141db83d8fd0cbc8fab13213187fc1469da3..52202c7df56aa1ec6c49df81007218448340fe69 100644 (file)
--- a/src/bin/pg_dump/pg_backup_archiver.c
+++ b/src/bin/pg_dump/pg_backup_archiver.c
@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *     $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.65 2003/01/13 04:28:55 inoue Exp $
+ *     $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.66 2003/01/27 00:23:38 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1379,14 +1379,15 @@ static void
 _die_horribly(ArchiveHandle *AH, const char *modulename, const char *fmt, va_list ap)
 {
    _write_msg(modulename, fmt, ap);
-   if (AH->public.verbose)
-       write_msg(NULL, "*** aborted because of error\n");
 
-   if (AH)
+   if (AH) {
+       if (AH->public.verbose)
+           write_msg(NULL, "*** aborted because of error\n");
        if (AH->connection)
            PQfinish(AH->connection);
-   if (AH->blobConnection)
-       PQfinish(AH->blobConnection);
+       if (AH->blobConnection)
+           PQfinish(AH->blobConnection);
+   }
 
    exit(1);
 }