mirror of
https://github.com/postgres/postgres.git
synced 2026-03-07 15:50:44 -05:00
868b39a544
Solaris has never bothered to add "const" to the second argument of PAM conversation procs, as all other Unixen did decades ago. This resulted in an "incompatible pointer" compiler warning when building --with-pam, but had no more serious effect than that, so we never did anything about it. However, as of GCC 14 the case is an error not warning by default. To complicate matters, recent OpenIndiana (and maybe illumos in general?) *does* supply the "const" by default, so we can't just assume that platforms using our solaris template need help. What we can do, short of building a configure-time probe, is to make solaris.h #define _PAM_LEGACY_NONCONST, which causes OpenIndiana's pam_appl.h to revert to the traditional definition, and hopefully will have no effect anywhere else. Then we can use that same symbol to control whether we include "const" in the declaration of pam_passwd_conv_proc(). Bug: #18995 Reported-by: Andrew Watkins <awatkins1966@gmail.com> Author: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/18995-82058da9ab4337a7@postgresql.org Backpatch-through: 13 |
||
|---|---|---|
| .. | ||
| atomics | ||
| win32 | ||
| win32_msvc | ||
| aix.h | ||
| atomics.h | ||
| cygwin.h | ||
| darwin.h | ||
| freebsd.h | ||
| hpux.h | ||
| linux.h | ||
| netbsd.h | ||
| openbsd.h | ||
| pg_bitutils.h | ||
| pg_bswap.h | ||
| pg_crc32c.h | ||
| pg_iovec.h | ||
| pg_pthread.h | ||
| solaris.h | ||
| win32.h | ||
| win32_port.h | ||
| win32ntdll.h | ||