Code review fix.

row.go

@@ -65,7 +65,22 @@ func (r *Row) scanCurrentRow() error {
 		return createError(fmt.Errorf("unknown columns to map to: %v", r.unknownDestinations), r.unknownDestinations...)
 	}
 
-	targets := makeTargets(len(r.columns))
+	targets, err := r.createTargets()
+	if err != nil {
+		return err
+	}
+
+	err = r.r.Scan(targets...)
+	if err != nil {
+		return err
+	}
+
+	r.scanDestinations = make([]reflect.Value, len(r.columns))
+	return nil
+}
+
+func (r *Row) createTargets() ([]any, error) {
+	targets := make([]any, len(r.columns))
 
 	for i, name := range r.columns {
 		dest := r.scanDestinations[i]
@@ -76,26 +91,14 @@ func (r *Row) scanCurrentRow() error {
 
 		if !r.allowUnknown {
 			err := fmt.Errorf("No destination for column %s", name)
-			return createError(err, "no destination", name)
+			return nil, createError(err, "no destination", name)
 		}
-	}
-
-	err := r.r.Scan(targets...)
-	if err != nil {
-		return err
-	}
 
-	r.scanDestinations = make([]reflect.Value, len(r.columns))
-	return nil
-}
-
-// See https://github.com/golang/go/issues/41607:
-// Some drivers cannot work with nil values, so valid pointers should be
-// used for all column targets, even if they are discarded afterwards.
-func makeTargets(length int) []any {
-	targets := make([]any, length)
-	for i := range targets {
+		// See https://github.com/golang/go/issues/41607:
+		// Some drivers cannot work with nil values, so valid pointers should be
+		// used for all column targets, even if they are discarded afterwards.
 		targets[i] = new(interface{})
 	}
-	return targets
+
+	return targets, nil
 }