Skip to content

Conversation

@JacobBarthelmeh
Copy link
Contributor

ZD16606

@JacobBarthelmeh JacobBarthelmeh self-assigned this Apr 15, 2025
@JacobBarthelmeh JacobBarthelmeh force-pushed the pkcs7_stream branch 2 times, most recently from 5195621 to 9f3dff5 Compare September 30, 2025 16:00
@JacobBarthelmeh
Copy link
Contributor Author

Retest this please Jenkins

@JacobBarthelmeh
Copy link
Contributor Author

Retest this please Jenkins. Valgrind report that I'm not sure why was parsed and flagged as an error all heap was free'd and I did not see any obvious errors listed :

Testing with configure --enable-rsa --enable-tls13

...
...
trying client command line[808]: SuiteTest -v 4 -l TLS13-AES128-GCM-SHA256 -x -2 -H exitWithRet -p 35801 
Skip exit() for testing
SSL version is TLSv1.3
SSL cipher suite is TLS_AES_128_GCM_SHA256
SSL curve name is FFDHE_2048
 Success -- All results as expected.
 End Cipher Suite Tests
unit_test: Success for all configured tests.
==1882914== 
==1882914== HEAP SUMMARY:
==1882914==     in use at exit: 0 bytes in 0 blocks
==1882914==   total heap usage: 85,359 allocs, 85,359 frees, 96,333,122 bytes allocated
==1882914== 
==1882914== All heap blocks were freed -- no leaks are possible
==1882914== 
==1882914== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
Exiting with status: 3
Script ran for 1775 seconds.
script returned exit code 3

@JacobBarthelmeh
Copy link
Contributor Author

Retest this please Jenkins

Found unhandled hudson.remoting.RequestAbortedException exception

@JacobBarthelmeh
Copy link
Contributor Author

Retest this please Jenkins.

FIPSv2 OE test configure error:
configure: error: unrecognized options: --with-libtool-sysroot
I don't think these PKCS7 changes would affect a FIPSv2 configure option. Maybe a unit test failure could happen from the changes but not a configure error.

@dgarske dgarske merged commit 92a4782 into wolfSSL:master Oct 6, 2025
334 of 336 checks passed
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.

3 participants