Check for non-existant connection in prepare statement handling.
authorMichael Meskes <meskes@postgresql.org>
Mon, 12 May 2008 16:29:04 +0000 (16:29 +0000)
committerMichael Meskes <meskes@postgresql.org>
Mon, 12 May 2008 16:29:04 +0000 (16:29 +0000)
Do not close files that weren't opened.

src/interfaces/ecpg/ChangeLog
src/interfaces/ecpg/ecpglib/prepare.c
src/interfaces/ecpg/preproc/preproc.y

index a5eb10b351ac800e8734979990e63bc3bed96b04..0572925d08d4abad4bac58e51190e439d6e144b8 100644 (file)
@@ -2346,6 +2346,11 @@ Thu, 10 Apr 2008 12:42:25 +0200
 
        - Fixed bug in PGTYPEStimestamp_sub that used pointers instead of the
          values to substract.
+
+Mon, 12 May 2008 18:19:08 +0200
+
+       - Check for non-existant connection in prepare statement handling.
+       - Do not close files that weren't opened.
        - Set pgtypes library version to 3.1.
        - Set compat library version to 3.1.
        - Set ecpg library version to 6.2.
index e1cc60ee19fc38d0242d4709aedd80994b6194ce..f20c7e7bad688f56217e6d35b4aad76e4b15cd60 100644 (file)
@@ -120,10 +120,11 @@ ECPGprepare(int lineno, const char *connection_name, const int questionmarks, co
        struct sqlca_t *sqlca = ECPGget_sqlca();
        PGresult   *query;
 
-       ecpg_init_sqlca(sqlca);
-
        con = ecpg_get_connection(connection_name);
 
+       if (!ecpg_init(con, connection_name, lineno))
+               return false;
+
        /* check if we already have prepared this statement */
        this = find_prepared_statement(name, con, &prev);
        if (this && !deallocate_one(lineno, ECPG_COMPAT_PGSQL, con, prev, this))
@@ -256,6 +257,9 @@ ECPGdeallocate(int lineno, int c, const char *connection_name, const char *name)
 
        con = ecpg_get_connection(connection_name);
 
+       if (!ecpg_init(con, connection_name, lineno))
+               return false;
+
        this = find_prepared_statement(name, con, &prev);
        if (this)
                return deallocate_one(lineno, c, con, prev, this);
index a43974f42acc544e4fac94784a6db342e859bb76..56ff85c013a8cdb4eb8c0064bc5cd64f17248936 100644 (file)
@@ -87,8 +87,10 @@ mmerror(int error_code, enum errortype type, char * error, ...)
                        ret_value = error_code;
                        break;
                case ET_FATAL:
-                       fclose(yyin);
-                       fclose(yyout);
+                       if (yyin)
+                               fclose(yyin);
+                       if (yyout)
+                               fclose(yyout);
                        if (unlink(output_filename) != 0 && *output_filename != '-')
                                fprintf(stderr, "Could not remove output file %s!\n", output_filename);
                        exit(error_code);