]> git.baikalelectronics.ru Git - arm-tf.git/commit
refactor(auth): use a single function for parsing extensions
authorDemi Marie Obenour <demiobenour@gmail.com>
Sat, 28 Jan 2023 20:15:37 +0000 (15:15 -0500)
committerDemi Marie Obenour <demiobenour@gmail.com>
Thu, 2 Mar 2023 16:48:43 +0000 (11:48 -0500)
commita987b89dab15ab7e79063aa085e696dcdb6feffe
tree36b84cb272762812acc60a2fb72ce831661bf972
parent74c4ae9cceda06177b9edaf8e057b6e45941c021
refactor(auth): use a single function for parsing extensions

Previously, extensions were parsed twice: once with error checking for
validation, and a second time without error checking to extract the
extension data.  This is error prone and caused TFV-10 (CVE-2022-47630).

A simpler approach is to have get_ext() be responsible for all extension
parsing, and to treat a NULL OID as an indicator that get_ext() is only
being called for validation.  cert_parse() checks that get_ext() returns
IMG_PARSER_OK and fails otherwise.

Change-Id: I65a2ff053a188351ba54799827a2b7bd833bb037
Signed-off-by: Demi Marie Obenour <demiobenour@gmail.com>
drivers/auth/mbedtls/mbedtls_x509_parser.c