#925 Fix using native client, password is limited to 255 bytes

Author: mrotteveel

src/docs/asciidoc/release_notes.adoc

@@ -50,6 +50,7 @@ This results in two minor breaking changes:
 ** Presence of the NUL character (U+0000) in an identifier passed to `enquoteIdentifier` will result in a `SQLSyntaxErrorException`
 * Fixed: JDBC escapes should not be parsed inside dialect 3 delimited identifiers or dialect 1 string literals (https://github.com/FirebirdSQL/jaybird/issues/921[#921])
 * Fixed: `IndexOutOfBoundsException` in `FBCachedBlob.getBytes(long, int)` for position or length beyond end of data (https://github.com/FirebirdSQL/jaybird/issues/923[#923])
+* Fixed: Using native client, password is limited to 255 bytes (https://github.com/FirebirdSQL/jaybird/issues/925[#925])
 
 === Jaybird 6.0.4
 

src/main/org/firebirdsql/gds/impl/DatabaseParameterBufferImp.java

@@ -55,7 +55,17 @@ public ParameterTagMapping getTagMapping() {
     }
 
     public enum DpbMetaData implements ParameterBufferMetaData {
-        DPB_VERSION_1(ISCConstants.isc_dpb_version1, ArgumentType.TraditionalDpb),
+        DPB_VERSION_1(ISCConstants.isc_dpb_version1, ArgumentType.TraditionalDpb) {
+            @Override
+            public boolean isUpgradable() {
+                return true;
+            }
+
+            @Override
+            public ParameterBufferMetaData upgradeMetaData() {
+                return DPB_VERSION_2;
+            }
+        },
         DPB_VERSION_2(ISCConstants.isc_dpb_version2, ArgumentType.Wide);
 
         private final int dpbVersion;

src/main/org/firebirdsql/gds/impl/ParameterBufferBase.java

@@ -50,7 +50,7 @@ public abstract class ParameterBufferBase implements ParameterBuffer, Serializab
     private final List<Argument> arguments = new ArrayList<>();
 
     private final String defaultEncodingName;
-    private final ParameterBufferMetaData parameterBufferMetaData;
+    private ParameterBufferMetaData parameterBufferMetaData;
     private transient Encoding defaultEncoding;
 
     /**
@@ -99,8 +99,21 @@ public final void addArgument(int argumentType, String value) {
 
     @Override
     public final void addArgument(int argumentType, String value, Encoding encoding) {
-        addArgument(new StringArgument(
-                argumentType, parameterBufferMetaData.getStringArgumentType(argumentType), value, encoding));
+        addArgument(createStringArgument(argumentType, value, encoding));
+    }
+
+    private StringArgument createStringArgument(int argumentType, String value, Encoding encoding) {
+        try {
+            return new StringArgument(
+                    argumentType, parameterBufferMetaData.getStringArgumentType(argumentType), value, encoding);
+        } catch (LengthOverflowException e) {
+            if (tryUpgradeMetaData()) {
+                // Do not inline parameterBufferMetaData or argument type; they might be different here than in the try
+                return new StringArgument(
+                        argumentType, parameterBufferMetaData.getStringArgumentType(argumentType), value, encoding);
+            }
+            throw e;
+        }
     }
 
     @Override
@@ -127,7 +140,19 @@ public final void addArgument(int argumentType) {
 
     @Override
     public final void addArgument(int type, byte[] content) {
-        addArgument(new ByteArrayArgument(type, parameterBufferMetaData.getByteArrayArgumentType(type), content));
+        addArgument(createByteArrayArgument(type, content));
+    }
+
+    private ByteArrayArgument createByteArrayArgument(int type, byte[] content) {
+        try {
+            return new ByteArrayArgument(type, parameterBufferMetaData.getByteArrayArgumentType(type), content);
+        } catch (LengthOverflowException e) {
+            if (tryUpgradeMetaData()) {
+                // Do not inline parameterBufferMetaData or argument type; they might be different here than in the try
+                return new ByteArrayArgument(type, parameterBufferMetaData.getByteArrayArgumentType(type), content);
+            }
+            throw e;
+        }
     }
 
     protected final void addArgument(Argument argument) {
@@ -227,6 +252,22 @@ public final boolean isEmpty() {
         return arguments.isEmpty();
     }
 
+    private boolean tryUpgradeMetaData() {
+        if (!parameterBufferMetaData.isUpgradable()) return false;
+        try {
+            ParameterBufferMetaData newParameterBufferMetaData = parameterBufferMetaData.upgradeMetaData();
+            List<Argument> newArguments = arguments.stream()
+                    .map(argument -> argument.transformTo(newParameterBufferMetaData))
+                    .toList();
+            arguments.clear();
+            arguments.addAll(newArguments);
+            parameterBufferMetaData = newParameterBufferMetaData;
+            return true;
+        } catch (IllegalArgumentException e) {
+            return false;
+        }
+    }
+
     @Override
     @SuppressWarnings("java:S2097")
     public final boolean equals(Object other) {

src/main/org/firebirdsql/gds/impl/ParameterBufferMetaData.java

@@ -108,4 +108,31 @@ default ArgumentType getByteArgumentType(int tag) {
         return getIntegerArgumentType(tag);
     }
 
+    /**
+     * Returns if this metadata type is upgradable.
+     * <p>
+     * The default implementation always returns {@code false}.
+     * </p>
+     *
+     * @return {@code true} if this type is upgradable
+     * @see #upgradeMetaData()
+     * @since 6.0.5
+     */
+    default boolean isUpgradable() {
+        return false;
+    }
+
+    /**
+     * The parameter buffer metadata that can be upgraded to.
+     * <p>
+     * The default implementation always returns {@code this}.
+     * </p>
+     *
+     * @return the upgrade buffer metadata, returns this instance if {@link #isUpgradable()} returns {@code false}
+     * @since 6.0.5
+     */
+    default ParameterBufferMetaData upgradeMetaData() {
+        return this;
+    }
+
 }

src/main/org/firebirdsql/gds/impl/ServiceParameterBufferImp.java

@@ -79,6 +79,16 @@ public ArgumentType getSingleArgumentType(int tag) {
                 }
                 return ArgumentType.TraditionalDpb;
             }
+
+            @Override
+            public boolean isUpgradable() {
+                return true;
+            }
+
+            @Override
+            public ParameterBufferMetaData upgradeMetaData() {
+                return SPB_VERSION_3_ATTACH;
+            }
         },
         // Technically this has nothing to do with SPB version 2/3
         SPB_VERSION_2(ISCConstants.isc_spb_current_version) {

src/main/org/firebirdsql/gds/impl/argument/Argument.java

@@ -19,6 +19,7 @@
 package org.firebirdsql.gds.impl.argument;
 
 import org.firebirdsql.gds.Parameter;
+import org.firebirdsql.gds.impl.ParameterBufferMetaData;
 
 import java.io.IOException;
 import java.io.OutputStream;
@@ -87,4 +88,17 @@ public long getValueAsLong() {
      */
     public abstract int getLength();
 
+    /**
+     * If needed, returns a new argument to transform to a suitable argument type, otherwise returns this instance.
+     *
+     * @param parameterBufferMetaData
+     *         parameter buffer metadata (used to determine the needed argument type)
+     * @return either a new argument if transformation is needed, or this instance
+     * @throws IllegalArgumentException
+     *         if transformation is needed, but the creation of the new instance fails (e.g. downgrading from
+     *         {@link ArgumentType#Wide} to {@link ArgumentType#TraditionalDpb}, and the value is too long)
+     * @since 6.0.5
+     */
+    public abstract Argument transformTo(ParameterBufferMetaData parameterBufferMetaData);
+
 }

src/main/org/firebirdsql/gds/impl/argument/BigIntArgument.java

@@ -21,6 +21,7 @@
 import org.firebirdsql.encodings.Encoding;
 import org.firebirdsql.gds.ParameterBuffer;
 import org.firebirdsql.gds.VaxEncoding;
+import org.firebirdsql.gds.impl.ParameterBufferMetaData;
 
 import java.io.IOException;
 import java.io.OutputStream;
@@ -82,6 +83,13 @@ public void copyTo(ParameterBuffer buffer, Encoding encoding) {
         buffer.addArgument(getType(), value);
     }
 
+    @Override
+    public BigIntArgument transformTo(ParameterBufferMetaData parameterBufferMetaData) {
+        ArgumentType newArgumentType = parameterBufferMetaData.getIntegerArgumentType(getType());
+        if (newArgumentType == argumentType) return this;
+        return new BigIntArgument(getType(), newArgumentType, value);
+    }
+
     @Override
     public boolean equals(Object other) {
         if (this == other) return true;

src/main/org/firebirdsql/gds/impl/argument/ByteArgument.java

@@ -20,6 +20,7 @@
 
 import org.firebirdsql.encodings.Encoding;
 import org.firebirdsql.gds.ParameterBuffer;
+import org.firebirdsql.gds.impl.ParameterBufferMetaData;
 
 import java.io.IOException;
 import java.io.OutputStream;
@@ -79,6 +80,13 @@ public void copyTo(ParameterBuffer buffer, Encoding encoding) {
         buffer.addArgument(getType(), value);
     }
 
+    @Override
+    public ByteArgument transformTo(ParameterBufferMetaData parameterBufferMetaData) {
+        ArgumentType newArgumentType = parameterBufferMetaData.getByteArgumentType(getType());
+        if (newArgumentType == argumentType) return this;
+        return new ByteArgument(getType(), newArgumentType, value);
+    }
+
     @Override
     public boolean equals(Object other) {
         if (this == other) return true;

src/main/org/firebirdsql/gds/impl/argument/ByteArrayArgument.java

@@ -20,6 +20,7 @@
 
 import org.firebirdsql.encodings.Encoding;
 import org.firebirdsql.gds.ParameterBuffer;
+import org.firebirdsql.gds.impl.ParameterBufferMetaData;
 
 import java.io.IOException;
 import java.io.OutputStream;
@@ -40,12 +41,18 @@ public final class ByteArrayArgument extends TypedArgument {
     private final byte[] value;
 
     /**
-     * Initializes an instance of ByteArrayArgument.
+     * Initialises an instance of ByteArrayArgument.
      *
      * @param type
-     *        Parameter type
+     *         parameter type
+     * @param argumentType
+     *         argument type
      * @param value
-     *        Byte array with a maximum length defined by {@code argumentType}.
+     *         byte array
+     * @throws IllegalArgumentException
+     *         if {@code type} is not valid for byte arrays, or if {@code value} is {@code null}
+     * @throws LengthOverflowException
+     *         if the length of {@code value} exceeds {@link ArgumentType#getMaxLength()}
      */
     public ByteArrayArgument(int type, ArgumentType argumentType, byte[] value) {
         super(type, argumentType);
@@ -58,7 +65,7 @@ public ByteArrayArgument(int type, ArgumentType argumentType, byte[] value) {
             throw new IllegalArgumentException("byte array value should not be null");
         }
         if (value.length > argumentType.getMaxLength()) {
-            throw new IllegalArgumentException(
+            throw new LengthOverflowException(
                     String.format("byte array value should not be longer than %d bytes, length was %d",
                             argumentType.getMaxLength(), value.length));
         }
@@ -86,11 +93,23 @@ public int getValueAsInt() {
         }
     }
 
+    // primarily intended for testing purposes
+    public byte[] getValueAsBytes() {
+        return value.clone();
+    }
+
     @Override
     public void copyTo(ParameterBuffer buffer, Encoding encoding) {
         buffer.addArgument(getType(), value.clone());
     }
 
+    @Override
+    public ByteArrayArgument transformTo(ParameterBufferMetaData parameterBufferMetaData) {
+        ArgumentType newArgumentType = parameterBufferMetaData.getByteArrayArgumentType(getType());
+        if (newArgumentType == argumentType) return this;
+        return new ByteArrayArgument(getType(), newArgumentType, value);
+    }
+
     @Override
     public boolean equals(Object other) {
         if (this == other) return true;

src/main/org/firebirdsql/gds/impl/argument/LengthOverflowException.java

@@ -0,0 +1,22 @@
+// SPDX-FileCopyrightText: Copyright 2026 Mark Rotteveel
+// SPDX-License-Identifier: LGPL-2.1-or-later
+package org.firebirdsql.gds.impl.argument;
+
+import java.io.Serial;
+
+/**
+ * Thrown by {@link StringArgument} and {@link ByteArrayArgument} if a value's length exceeds that of their
+ * {@link ArgumentType}.
+ *
+ * @since 6.0.5
+ */
+public class LengthOverflowException extends IllegalArgumentException {
+
+    @Serial
+    private static final long serialVersionUID = 5735116134576091931L;
+
+    LengthOverflowException(String message) {
+        super(message);
+    }
+
+}