diff options
author | Thomas Heijligen <thomas.heijligen@secunet.de> | 2021-06-09 11:50:17 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2021-06-20 09:10:15 +0000 |
commit | 6aaa51f19a826f18ac19ccf2acf98435d3f52ce7 (patch) | |
tree | 65477cabf987a83ef88c2a746a948e0eef7e3b05 /libflashrom.c | |
parent | d3ee782e4b3f311dfd0d58427db94aaf5436d928 (diff) | |
download | flashrom-6aaa51f19a826f18ac19ccf2acf98435d3f52ce7.tar.gz flashrom-6aaa51f19a826f18ac19ccf2acf98435d3f52ce7.tar.bz2 flashrom-6aaa51f19a826f18ac19ccf2acf98435d3f52ce7.zip |
libflashrom: remove flashrom_supported_programmers
const char **flashrom_supported_programmers(void) returns an array of
strings without returning the array size or making a NULL
termination. This can lead to undefined behavior when iterating over the
array.
Change-Id: I0157926a654e337c14d840dd398e5576471c304f
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.de>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/55350
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'libflashrom.c')
-rw-r--r-- | libflashrom.c | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/libflashrom.c b/libflashrom.c index 67d3ef307..d999efcd0 100644 --- a/libflashrom.c +++ b/libflashrom.c @@ -114,26 +114,6 @@ const char *flashrom_version_info(void) } /** - * @brief Returns list of supported programmers - * @return List of supported programmers, or NULL if an error occurred - */ -const char **flashrom_supported_programmers(void) -{ - size_t p = 0; - const char **supported_programmers = malloc((programmer_table_size + 1) * sizeof(char*)); - - if (supported_programmers != NULL) { - for (; p < programmer_table_size; ++p) { - supported_programmers[p] = programmer_table[p]->name; - } - } else { - msg_gerr("Memory allocation error!\n"); - } - - return supported_programmers; -} - -/** * @brief Returns list of supported flash chips * @return List of supported flash chips, or NULL if an error occurred */ |