Re: Clang 3.3 Analyzer Results

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: Clang 3.3 Analyzer Results
Дата
Msg-id 20131113001108.GA16066@eldon.alvh.no-ip.org
обсуждение исходный текст
Ответ на Re: Clang 3.3 Analyzer Results  (Kevin Grittner <kgrittn@ymail.com>)
Ответы Re: Clang 3.3 Analyzer Results  (Jeffrey Walton <noloader@gmail.com>)
Список pgsql-hackers
Kevin Grittner escribió:

> 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.

We have marked a large number of memory leak reports by Coverity in
initdb and other short-lived programs as false positive, on the grounds
that there's no point in freeing memory in a program that's about to
terminate anyway.  I'm not saying I agree necessarily with that POV, but
if we take that stance then there's similarly no point in fixing this
leak in the regression test code, is there?

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andreas
Дата:
Сообщение: Re: Relax table alias conflict rule in 9.3?
Следующее
От: Jeffrey Walton
Дата:
Сообщение: Re: Clang 3.3 Analyzer Results