Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use libxml2/xmsec1 for XAdES signature parsing and validation #607

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

metsma
Copy link
Contributor

@metsma metsma commented Jul 18, 2024

IB-7948

Signed-off-by: Raul Metsma [email protected]

src/SignatureXAdES_B.cpp Fixed Show fixed Hide fixed
src/SignatureXAdES_B.cpp Fixed Show fixed Hide fixed
@metsma metsma force-pushed the xades branch 4 times, most recently from 4a2a0c7 to aaabaae Compare July 22, 2024 19:59
src/Container.cpp Fixed Show fixed Hide fixed
@metsma metsma force-pushed the xades branch 7 times, most recently from 75adc5f to df35be2 Compare July 24, 2024 12:44
src/SignatureXAdES_B.cpp Fixed Show fixed Hide fixed
@metsma metsma force-pushed the xades branch 10 times, most recently from 2a9a96c to b3dc046 Compare August 1, 2024 07:16
src/SignatureXAdES_B.cpp Fixed Show fixed Hide fixed
@metsma metsma force-pushed the xades branch 4 times, most recently from b19e45e to 2f978da Compare August 1, 2024 12:56
@metsma metsma force-pushed the xades branch 2 times, most recently from 324d5b6 to bbf596b Compare August 1, 2024 13:03
src/XMLDocument.h Fixed Show fixed Hide fixed
{

cb_doc = bdoc;
cb_exception = &exception;

Check warning

Code scanning / CodeQL

Local variable address stored in non-local memory Warning

A stack address (
source
) may be assigned to a non-local variable.
@metsma metsma force-pushed the xades branch 4 times, most recently from a5572a1 to 49ec4f5 Compare August 5, 2024 11:28
@kristelmerilain kristelmerilain merged commit 79d7001 into open-eid:master Aug 6, 2024
38 checks passed
@metsma metsma deleted the xades branch August 6, 2024 19:17
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.

2 participants