Re: BUG #18889: /src/backend/utils/adt/network.c condition is always false

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: dan-eli(at)mail(dot)ru
Cc: pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #18889: /src/backend/utils/adt/network.c condition is always false
Date: 2025-04-10 18:19:52
Message-ID: 593243.1744309192@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

PG Bug reporting form <noreply(at)postgresql(dot)org> writes:
> On line 282 there is a check of nb < 0, which is always false, so i think it
> is better to be replaced with 'Assert(nb > 0);'

This one I do agree with getting rid of --- in fact, I don't
even see a need for an Assert. Done at f27eb0325.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Robins Tharakan 2025-04-11 12:48:04 Re: ERROR: tuple to be updated was already modified by an operation triggered by the current command
Previous Message Euler Taveira 2025-04-10 18:17:17 Re: BUG #18888: /src/interfaces/ecpg/preproc/descriptor.c possible NULL dereference