go-playground / validator

74%

v8: 100%

LAST BUILD ON BRANCH fix-unique-nil-panic

branch: fix-unique-nil-panic

CHANGE BRANCH

x

Reset

• fix-unique-nil-panic
• #1519/feature/trx_addr-tag
• FIX_FILEPATH_TAG
• add-dependabot
• add-fl-field-name
• add-isdefault
• add-support-to-tag-valid
• add-unique
• add_alpha_space_validator
• bcp47_strict_language_tag
• bic-iso-9362-2022
• bump_x_net
• bux-unix-addr-is-useless
• cherry-pick-v9
• correct-var-lock
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/setup-go-6
• dependabot/github_actions/golangci/golangci-lint-action-6
• dependabot/github_actions/golangci/golangci-lint-action-7
• dependabot/github_actions/golangci/golangci-lint-action-8
• dependabot/github_actions/golangci/golangci-lint-action-9
• dependabot/go_modules/_examples/validate_fn/golang.org/x/crypto-0.35.0
• dependabot/go_modules/_examples/validate_fn/golang.org/x/crypto-0.45.0
• dependabot/go_modules/_examples/validate_fn/golang.org/x/net-0.38.0
• dependabot/go_modules/github.com/gabriel-vasile/mimetype-1.4.10
• dependabot/go_modules/github.com/gabriel-vasile/mimetype-1.4.11
• dependabot/go_modules/github.com/gabriel-vasile/mimetype-1.4.12
• dependabot/go_modules/github.com/gabriel-vasile/mimetype-1.4.13
• dependabot/go_modules/github.com/gabriel-vasile/mimetype-1.4.9
• dependabot/go_modules/golang.org/x/crypto-0.33.0
• dependabot/go_modules/golang.org/x/crypto-0.38.0
• dependabot/go_modules/golang.org/x/crypto-0.39.0
• dependabot/go_modules/golang.org/x/crypto-0.40.0
• dependabot/go_modules/golang.org/x/crypto-0.41.0
• dependabot/go_modules/golang.org/x/crypto-0.42.0
• dependabot/go_modules/golang.org/x/crypto-0.43.0
• dependabot/go_modules/golang.org/x/crypto-0.45.0
• dependabot/go_modules/golang.org/x/crypto-0.46.0
• dependabot/go_modules/golang.org/x/crypto-0.47.0
• dependabot/go_modules/golang.org/x/crypto-0.48.0
• dependabot/go_modules/golang.org/x/net-0.38.0
• dependabot/go_modules/golang.org/x/text-0.22.0
• dependabot/go_modules/golang.org/x/text-0.25.0
• dependabot/go_modules/golang.org/x/text-0.26.0
• dependabot/go_modules/golang.org/x/text-0.27.0
• dependabot/go_modules/golang.org/x/text-0.28.0
• dependabot/go_modules/golang.org/x/text-0.29.0
• dependabot/go_modules/golang.org/x/text-0.30.0
• dependabot/go_modules/golang.org/x/text-0.31.0
• dependabot/go_modules/golang.org/x/text-0.32.0
• dependabot/go_modules/golang.org/x/text-0.33.0
• dependabot/go_modules/golang.org/x/text-0.34.0
• doc/validation-error-filed
• docs/fix-typos
• document-omitzero
• duration-translation-en-fix
• email-regexp-optimization
• en_urn_rfc2141
• enable-more-linters
• example-simple-signup
• exclude-modernize-linter
• feat/add-translation-de-alphanumspace-alphanumunicode
• feat/improve-indonesian-translations-and-add-tests
• feat/option-multi-error-returned
• feat/urn-8141
• feat/validation-valuer
• feature/add-cmyk-to-validator
• feature/add-early-return-error
• feature/german-translations
• feature/omitzero
• field-level-validation2
• fix-1500
• fix-1502-update-ccy-codes
• fix-1513-iso31662-codes
• fix-32bit-test-failures
• fix-consequtive-ors
• fix-cron-validation
• fix-duration
• fix-e164-regexp
• fix-examples-errors-as
• fix-linter-errors-2.0.2
• fix-map-validation-missing-keys
• fix-required-without
• fix-rfc-1035-label-length-check
• fix-unique-map-nil-values
• fix/1481-skip-invalid-type-validations
• fix/add-missing-alpha-translations-1480
• fix/alias-with-or-operator-766
• fix/remove-issue-template-md
• fix/should-panic-required-if-duplicate-fieldname
• fix/update-mimetype
• fix/validatemap-cross-field-panic-893
• fix_fqdn
• fixes
• general-maint
• general-maint2
• github-action-support-2-major-go-versions
• github-actions
• github-workflow-msgv
• go-1.25
• go-context-support
• heads/v5
• heads/v7
• heads/v8
• https_url
• i18n-custom-errors
• issue-1306-excluded_unless-expectation-parity-with-required_unless
• issue-743
• jmfrees/fix-required_without
• krhubert
• map-struct-validation-bug
• master
• minor-perf-1
• minor-perf-2
• msgv-changes
• omitzero-slice-map-bug
• paxman23l/fail-fast-feature
• port-validator-doc
• readme-fix-release-version-badge
• replace-travis-ci-badge-with-gha
• require-check-field-kind-unparam
• revert-e164-min-limit
• rm-gitter-im-link
• s-existant-existent
• stricter-unicode-space-check-in-email-regex
• supplyChainHardening
• translate-thai
• translation
• unicode-alphas
• unit-tests-startsnotwith
• unnecessary-statement
• upd-readme-prj-status-release-label
• update-deps-dec-2024
• update-docs
• updates-and-fixes
• url_file
• urls
• v10.0.0
• v10.0.1
• v10.1.0
• v10.2.0
• v10.3.0
• v10.4.0
• v10.4.1
• v5
• v5-development
• v5.10.1
• v5.10.2
• v6
• v6-development
• v7
• v7-development
• v8
• v8-backport-test-fixes
• v8-development
• v9
• v9.30.1
• validate-keys
• validate-map-keys
• validator-panic-bug

Build # 21856889411

Build Type

Pull #1530

github

Committed by veeceey

Commit Message

fix: prevent panic in unique validation with nil pointer elements

The `unique` tag panics when validating slices or maps containing nil
pointer elements. This affects three code paths in isUnique: slices
without a field param, slices with a field param (unique=Field), and
maps with pointer values.

The fix safely skips nil elements after reflect.Indirect returns a zero
Value, and treats multiple nil entries as duplicates (not unique).

Fixes #1319

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Pull Request Pull Request #1530: fix: prevent panic in unique validation with nil pointer elements

Run Details

21 of 21 new or added lines in 1 file covered. (100.0%)

14470 of 19594 relevant lines covered (73.85%)

81.84 hits per line