[openssl/openssl] 709c04: punycode: update to use WPACKET instead of using c...

Pauli noreply at github.com
Thu Nov 10 21:24:56 UTC 2022


  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 709c04b5dd6a24f88459d9e214e85e396b2471fd
      https://github.com/openssl/openssl/commit/709c04b5dd6a24f88459d9e214e85e396b2471fd
  Author: Pauli <pauli at openssl.org>
  Date:   2022-11-11 (Fri, 11 Nov 2022)

  Changed paths:
    M crypto/punycode.c
    M crypto/x509/v3_ncons.c
    M doc/internal/man3/ossl_punycode_decode.pod
    M include/crypto/punycode.h
    M test/punycode_test.c

  Log Message:
  -----------
  punycode: update to use WPACKET instead of using custom range checking

Add test for `.' overflows, remove the output size argument from
ossl_a2ulabel() since it was never used and greatly complicated the code.
Convert ossl_a2ulabel() to use WPACKET for building the output string.
Update the documentation to match the new definition of ossl_a2ulabel().

x509: let punycode handle the '\0' string termination.  Saves a memset(3)
and some size fiddling.  Also update to deal with the modified parameters.

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit at gmail.com>
(Merged from https://github.com/openssl/openssl/pull/19591)

(cherry picked from commit 905ba924398f474e647de70345b4ae4089fedba7)


  Commit: f01ebab0c0ef26677ab6d885922ca1a1b24494fc
      https://github.com/openssl/openssl/commit/f01ebab0c0ef26677ab6d885922ca1a1b24494fc
  Author: Pauli <pauli at openssl.org>
  Date:   2022-11-11 (Fri, 11 Nov 2022)

  Changed paths:
    M fuzz/build.info
    A fuzz/corpora/punycode/0000000000000000000000000000000000000000
    A fuzz/corpora/punycode/0000000000000000000000000000000000000001
    M fuzz/fuzzer.h
    A fuzz/punycode.c
    M include/crypto/punycode.h

  Log Message:
  -----------
  fuzz: add punycode decoder fuzz test

Reviewed-by: Tomas Mraz <tomas at openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit at gmail.com>
(Merged from https://github.com/openssl/openssl/pull/19591)

(cherry picked from commit 8aa82b337081b7a22c35dddad8d62fb1ca9ea884)


Compare: https://github.com/openssl/openssl/compare/7abe06cbb52c...f01ebab0c0ef


More information about the openssl-commits mailing list