postgresql-10.0/src/backend/optimizer/geqo/geqo_ox2.c:84:sanity check after use ?

From: David Binderman <dcb314(at)hotmail(dot)com>
To: "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: postgresql-10.0/src/backend/optimizer/geqo/geqo_ox2.c:84:sanity check after use ?
Date: 2017-10-06 15:18:07
Message-ID: HE1PR0802MB2331A1955CFC7D1D2A6338F19C710@HE1PR0802MB2331.eurprd08.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Hello there,

postgresql-10.0/src/backend/optimizer/geqo/geqo_ox2.c:84]: (style) Array index 'j' is used before limits check.

Source code is

while ((city_table[j].select_list == -1) && (j < num_gene))

Suggest sanity check array index before use, not after.

Regards

David Binderman

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2017-10-06 15:37:05 Re: postgresql-10.0/src/bin/psql/mainloop.c:465: suspicious condition ?
Previous Message David Binderman 2017-10-06 15:15:46 postgresql-10.0/src/backend/access/transam/clog.c:234: sanity check after use ?