Skip to content

Fix SSL015 CANARIES condition#140

Open
Marsman1996 wants to merge 1 commit intoHexHive:devfrom
Marsman1996:dev
Open

Fix SSL015 CANARIES condition#140
Marsman1996 wants to merge 1 commit intoHexHive:devfrom
Marsman1996:dev

Conversation

@Marsman1996
Copy link
Copy Markdown

According to the code in MAGMA_ENABLE_FIXES

if (p7->d.ptr == NULL) {
    PKCS7err(PKCS7_F_PKCS7_DATAINIT, PKCS7_R_NO_CONTENT);
    return NULL;
}

and other 2 CANARIES code,

+ MAGMA_LOG("%MAGMA_BUG%", MAGMA_OR(p7 == NULL, p7->d.ptr == NULL));

+ MAGMA_LOG("%MAGMA_BUG%", MAGMA_OR(p7 == NULL, p7->d.ptr == NULL));

p7->d.ptr should also be null when vulnerability is triggered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant