Re: Small patch for memory leak in src/backend/catalog/pg_proc.c - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: Small patch for memory leak in src/backend/catalog/pg_proc.c
Date
Msg-id 200006140452.AAA23321@candle.pha.pa.us
Whole thread Raw
In response to Re: Small patch for memory leak in src/backend/catalog/pg_proc.c  (Denis Perchine <dyp@perchine.com>)
List pgsql-patches
Applied.

> > But anyway it is better to free memory, because transaction can be quite large.
> > New patch attached.
>
> :-((( Again small mistake.
>
> --
> Sincerely Yours,
> Denis Perchine
>
> ----------------------------------
> E-Mail: dyp@perchine.com
> HomePage: http://www.perchine.com/dyp/
> FidoNet: 2:5000/120.5
> ----------------------------------

[ Attachment, skipping... ]
Index: pg_proc.c
===================================================================
RCS file: /home/projects/pgsql/cvsroot/pgsql/src/backend/catalog/pg_proc.c,v
retrieving revision 1.43
diff -c -b -w -r1.43 pg_proc.c
*** pg_proc.c    2000/05/28 17:55:54    1.43
--- pg_proc.c    2000/06/13 15:31:57
***************
*** 67,72 ****
--- 67,73 ----
      Oid            toid;
      NameData    procname;
      TupleDesc    tupDesc;
+     Oid        retval;

      /* ----------------
       *    sanity checks
***************
*** 327,331 ****
          CatalogCloseIndices(Num_pg_proc_indices, idescs);
      }
      heap_close(rel, RowExclusiveLock);
!     return tup->t_data->t_oid;
  }
--- 328,334 ----
          CatalogCloseIndices(Num_pg_proc_indices, idescs);
      }
      heap_close(rel, RowExclusiveLock);
!     retval = tup->t_data->t_oid;
!     heap_freetuple(tup);
!     return retval;
  }


--
  Bruce Momjian                        |  http://www.op.net/~candle
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

pgsql-patches by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: odbc patches
Next
From: "David Reid"
Date:
Subject: BeOS and IPC - try 999