OSDN Git Service

Fix a couple of rather-pointless-but-easily-fixed Coverity warnings.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 19 Apr 2006 16:02:17 +0000 (16:02 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 19 Apr 2006 16:02:17 +0000 (16:02 +0000)
Per Martijn van Oosterhout.

src/bin/pg_dump/pg_backup_archiver.c
src/bin/psql/prompt.c

index e9fea7e..0f56a64 100644 (file)
@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.126 2006/04/12 22:18:48 tgl Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.127 2006/04/19 16:02:17 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2203,7 +2203,7 @@ _selectOutputSchema(ArchiveHandle *AH, const char *schemaName)
        PQExpBuffer qry;
 
        if (!schemaName || *schemaName == '\0' ||
-               strcmp(AH->currSchema, schemaName) == 0)
+               (AH->currSchema && strcmp(AH->currSchema, schemaName) == 0))
                return;                                 /* no need to do anything */
 
        qry = createPQExpBuffer();
index a11d849..f7d591b 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2006, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/prompt.c,v 1.43 2006/03/05 15:58:52 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/prompt.c,v 1.44 2006/04/19 16:02:17 tgl Exp $
  */
 #include "postgres_fe.h"
 #include "prompt.h"
@@ -250,14 +250,13 @@ get_prompt(promptStatus_t status)
                                        /* execute command */
                                case '`':
                                        {
-                                               FILE       *fd = NULL;
+                                               FILE       *fd;
                                                char       *file = pg_strdup(p + 1);
                                                int                     cmdend;
 
                                                cmdend = strcspn(file, "`");
                                                file[cmdend] = '\0';
-                                               if (file)
-                                                       fd = popen(file, "r");
+                                               fd = popen(file, "r");
                                                if (fd)
                                                {
                                                        fgets(buf, MAX_PROMPT_SIZE - 1, fd);