From 1ba0b3f6fe4c4e941ef40428567a3e20e64f8bc6 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Fri, 14 Feb 2014 16:06:07 +0100 Subject: [PATCH] [#3046] UDTRecordImpl.toString() may throw NullPointerException, if custom RecordMapperProvider maps Record to null --- jOOQ/src/main/java/org/jooq/impl/UDTRecordImpl.java | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/UDTRecordImpl.java b/jOOQ/src/main/java/org/jooq/impl/UDTRecordImpl.java index b542d51413..b4d054559f 100644 --- a/jOOQ/src/main/java/org/jooq/impl/UDTRecordImpl.java +++ b/jOOQ/src/main/java/org/jooq/impl/UDTRecordImpl.java @@ -148,11 +148,16 @@ public class UDTRecordImpl> extends AbstractRecord implem result.append(create().render(getUDT())); result.append("("); - for (Object o : intoArray()) { - result.append(separator); - result.append(o); + Object[] array = intoArray(); - separator = ", "; + // [#3046] array can be null if custom RecordMapperProviders (illegally) return null + if (array != null) { + for (Object o : array) { + result.append(separator); + result.append(o); + + separator = ", "; + } } result.append(")");