[Opensc-commits] [OpenSC/OpenSC] a285b3: Check nested macros when parsing pkcs15init profile

[Veronika H. <no...@gi...>]

  Branch: refs/heads/master
  Home:   https://github.com/OpenSC/OpenSC
  Commit: a285b300305ab19a706dac158695c8d92b96d1ff
      https://github.com/OpenSC/OpenSC/commit/a285b300305ab19a706dac158695c8d92b96d1ff
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/profile.c

  Log Message:
  -----------
  Check nested macros when parsing pkcs15init profile

When processing macros in pkcs15init profile, the macros
may contain nested macros, which causes recursion loop.

In this particular case found by OSS-Fuzz, the problem
was caused by macro e =1)$e

Thank you OSS-Fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=53545


  Commit: fd4754b101a50541d41216ba4d9d57bc9e2991cc
      https://github.com/OpenSC/OpenSC/commit/fd4754b101a50541d41216ba4d9d57bc9e2991cc
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/profile.c

  Log Message:
  -----------
  Handle errors when processing macros


  Commit: b6dc20b29ba5da4cac6ebb0a4539233e58fa5fcc
      https://github.com/OpenSC/OpenSC/commit/b6dc20b29ba5da4cac6ebb0a4539233e58fa5fcc
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/libopensc/muscle-filesystem.c
    M src/libopensc/muscle-filesystem.h

  Log Message:
  -----------
  muscle-filesystem: Do not process too deep filesystem

Given test-case causes the creation of too deep filesystem,
which processing recursive leads to stack-overflow.

Thanks OSS-Fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=54592


  Commit: f169e3f56b4e061267d188c66c7223ba12177211
      https://github.com/OpenSC/OpenSC/commit/f169e3f56b4e061267d188c66c7223ba12177211
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/libopensc/card-muscle.c

  Log Message:
  -----------
  Add error checking in muscle driver


  Commit: 5631e9843c832a99769def85b7b9b68b4e3e3959
      https://github.com/OpenSC/OpenSC/commit/5631e9843c832a99769def85b7b9b68b4e3e3959
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/profile.c

  Log Message:
  -----------
  Check length of string before making copy

Thanks OSS-Fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=55851
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=55998


  Commit: eac9f21202377cb2b34246f2920504dfbd92b139
      https://github.com/OpenSC/OpenSC/commit/eac9f21202377cb2b34246f2920504dfbd92b139
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/pkcs15-lib.c

  Log Message:
  -----------
  Fix file memory leak

Thanks OSS-Fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=56736


  Commit: d0a2bdaac5dcecc905c7aad6579c46624cc0ebde
      https://github.com/OpenSC/OpenSC/commit/d0a2bdaac5dcecc905c7aad6579c46624cc0ebde
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/pkcs15-oberthur.c

  Log Message:
  -----------
  Check length of path

Thanks OSS-Fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=56781


  Commit: 34f6ca6ed1cdd6eeb0faac4087634467ff604a26
      https://github.com/OpenSC/OpenSC/commit/34f6ca6ed1cdd6eeb0faac4087634467ff604a26
  Author: Veronika Hanulikova <xha...@fi...>
  Date:   2023-03-21 (Tue, 21 Mar 2023)

  Changed paths:
    M src/pkcs15init/profile.c

  Log Message:
  -----------
  Use better naming for macro-related variables


Compare: https://github.com/OpenSC/OpenSC/compare/5d3657f76b0f...34f6ca6ed1cd