Fix HBA option count
authorDaniel Gustafsson <dgustafsson@postgresql.org>
Sun, 12 Jan 2025 22:44:39 +0000 (23:44 +0100)
committerDaniel Gustafsson <dgustafsson@postgresql.org>
Sun, 12 Jan 2025 22:44:39 +0000 (23:44 +0100)
Commit 27a1f8d108 missed updating the max HBA option count to
account for the new option added.  Fix by bumping the counter
and adjust the relevant comment to match.  Backpatch down to
all supported branches like the erroneous commit.

Reported-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/286764.1736697356@sss.pgh.pa.us
Backpatch-through: v13

src/backend/utils/adt/hbafuncs.c

index 053221814e0f725a4d920baaa2bd93e5bef6cd49..ce1e2921ab3149f4989c86f388b4bdf64c4b640e 100644 (file)
@@ -38,12 +38,12 @@ static void fill_ident_view(Tuplestorestate *tuple_store, TupleDesc tupdesc);
 /*
  * This macro specifies the maximum number of authentication options
  * that are possible with any given authentication method that is supported.
- * Currently LDAP supports 11, and there are 3 that are not dependent on
+ * Currently LDAP supports 12, and there are 3 that are not dependent on
  * the auth method here.  It may not actually be possible to set all of them
  * at the same time, but we'll set the macro value high enough to be
  * conservative and avoid warnings from static analysis tools.
  */
-#define MAX_HBA_OPTIONS 14
+#define MAX_HBA_OPTIONS 15
 
 /*
  * Create a text array listing the options specified in the HBA line.