giomba
/
opensc
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

openpgp: Properly free the allocated file structure 

Resolves:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=16879
This commit is contained in:
Jakub Jelen 2019-10-01 11:00:06 +02:00 committed by Frank Morgner
parent 79cb753921
commit 34bd879400
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/libopensc/card-openpgp.c
View File

@ -513,6 +513,7 @@ pgp_init(sc_card_t *card)
/* explicitly get the full aid */
r = get_full_pgp_aid(card, file);
if (r < 0) {
sc_file_free(file);
pgp_finish(card);
LOG_FUNC_RETURN(card->ctx, SC_ERROR_INVALID_CARD);
}
@ -577,8 +578,10 @@ pgp_init(sc_card_t *card)
sc_format_path("3f00", &file->path);
/* set up the root of our fake file tree */
/* Transfers ownership of the file to the priv->mf structure */
priv->mf = pgp_new_blob(card, NULL, 0x3f00, file);
if (!priv->mf) {
sc_file_free(file);
pgp_finish(card);
LOG_FUNC_RETURN(card->ctx, SC_ERROR_OUT_OF_MEMORY);
}