| From: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
|---|---|
| To: | Kevin Grittner <kgrittn(at)ymail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Peter Geoghegan <pg(at)heroku(dot)com>, "noloader(at)gmail(dot)com" <noloader(at)gmail(dot)com>, Tomas Vondra <tv(at)fuzzy(dot)cz>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Clang 3.3 Analyzer Results |
| Date: | 2013-11-12 16:41:01 |
| Message-ID: | 1384274461.86356.YahooMailNeo@web162902.mail.bf1.yahoo.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-general pgsql-hackers |
Kevin Grittner <kgrittn(at)ymail(dot)com> wrote:
> Memory Error
> Double free: 1
> Memory leak: 1
These both seemed legitimate to me. Patch attached. Any
objections to applying it? I realize the memory leak is a tiny one
in the regression testing code, so it could never amount to enough
to matter; but it seems worth fixing just to avoid noise in code
analyzers.
--
Kevin Grittner
EDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| Attachment | Content-Type | Size |
|---|---|---|
| clang-memory-error-fixes-v1.diff | text/x-diff | 671 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2013-11-12 17:00:01 | Re: Clang 3.3 Analyzer Results |
| Previous Message | Brian Crowell | 2013-11-12 16:19:52 | Re: GSSAPI server side on Linux, SSPI client side on Windows |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Claudio Freire | 2013-11-12 16:45:19 | Re: What's needed for cache-only table scan? |
| Previous Message | J Smith | 2013-11-12 16:32:34 | Re: Errors on missing pg_subtrans/ files with 9.3 |