refactor SessionSecret_callback*

[effbiae]

Description

src/internal.c contains SessionSecret_callback and SessionSecret_callback_Tls13 which share a page of code.

this patch pulls out the common code to SessionSecret_callback_common

Testing

./configure --enable-opensslextra --enable-keylog-export --enable-tls13 --enable-all && make check

Checklist

• added tests
• updated/added doxygen
• updated appropriate READMEs
• Updated manual and documentation

[wolfSSL-Bot]

Can one of the admins verify this patch?

[devin-ai-integration]

🛟 Devin Lifeguard found 2 likely issues in this PR

• pointer-null-check snippet: Add if (ssl->ctx == NULL) return BAD_FUNC_ARG; immediately before the line logCb = ssl->ctx->keyLogCb; inside SessionSecret_callback_common().
• declare-const-pointers snippet: Change the signature of SessionSecret_callback_common to static int SessionSecret_callback_common(const WOLFSSL* ssl, const unsigned char* secret, int secretSz, const char* label, int labelSz) and adjust its callers accordingly.

@effbiae
please take a look at the above issues which Devin flagged. Devin will not fix these issues automatically.

[dgarske]

Okay to test. Contributor agreement on file.

[dgarske]

Okay to test. Contributor agreement on file.

[effbiae]

i've rebased on upstream/master to get the fix for the two failures