pgsql: Avoid platform-specific null pointer dereference in psql. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Avoid platform-specific null pointer dereference in psql.
Date
Msg-id E1i1xof-0001Da-1P@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Avoid platform-specific null pointer dereference in psql.

POSIX permits getopt() to advance optind beyond argc when the last
argv entry is an option that requires an argument and hasn't got one.
It seems that no major platforms actually do that, but musl does,
so that something like "psql -f" would crash with that libc.
Add a check that optind is in range before trying to look at the
possibly-bogus option.

Report and fix by Quentin Rameau.  Back-patch to all supported
branches.

Discussion: https://postgr.es/m/20190825100617.GA6087@fifth.space

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6338fa3e715ad1cb12e0760bf73ffc2a906098ea

Modified Files
--------------
src/bin/psql/startup.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Back off output precision in circle.sql regression test.
Next
From: Peter Geoghegan
Date:
Subject: Re: pgsql: Fix optimization of foreign-key on update actions