On Sun, Jun 4, 2023 at 8:42 PM Ranier Vilela <ranier.vf@gmail.com> wrote:Hi,Per Coverity.At function ExtendBufferedRelShared, has a always true test.eb.rel was dereferenced one line above, so in if (eb.rel) is always true.I think it's worth removing the test, because Coverity raises dozens of alerts thinking eb.rel might be NULL.Besides, one less test is one less branch.This also happens in ExtendBufferedRelTo, and the comment there explainsthat the eb.rel 'could have been closed while waiting for lock'.
Hi,Per Coverity.At function ExtendBufferedRelShared, has a always true test.eb.rel was dereferenced one line above, so in if (eb.rel) is always true.I think it's worth removing the test, because Coverity raises dozens of alerts thinking eb.rel might be NULL.Besides, one less test is one less branch.
pgsql-hackers by date:
Соглашаюсь с условиями обработки персональных данных