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

UserWarning: Verification of GCM tag failed: MAC check failed #163

Open
jurgonaut opened this issue Apr 2, 2020 · 1 comment
Open

UserWarning: Verification of GCM tag failed: MAC check failed #163

jurgonaut opened this issue Apr 2, 2020 · 1 comment

Comments

@jurgonaut
Copy link

I am seeing this warning:

/usr/local/lib/python3.7/dist-packages/scapy/layers/ssl_tls_crypto.py:1095: UserWarning: Verification of GCM tag failed: MAC check failed
  warnings.warn("Verification of GCM tag failed: %s" % why)

Firs I do a "do_handshake" on a TLSSoket, the handshake is successful and after that I do do_round_trip(TLSPlaintext(data=)). After the round trip the warning is printed. The program works fine I just want to know if the warning can be ignored or otherwise how to fix it.

Thanks in advance and for the great work you are doing with this module.

krizhanovsky added a commit to tempesta-tech/tempesta-test that referenced this issue Dec 7, 2020
(tintinweb/scapy-ssl_tls#163).

RFC 5246 6.2.3.3 requires the explicit nonce (most usually sequence
number) in the additional authenticated data (AAD), however
crypto_data.sequence, used for the AAD creation in EAEADCryptoContainer,
wasn't initialized with the explicit_nonce.
@krizhanovsky
Copy link

We encountered the problem (tempesta-tech/tempesta#1310) as well and fixed it with tempesta-tech/tempesta-test@51f2680 . I didn't open a pull request for this original repo since it's unsupported any more, but I hope the fix will be useful for other people facing the problem.

krizhanovsky added a commit to tempesta-tech/tempesta-test that referenced this issue Dec 11, 2020
(tintinweb/scapy-ssl_tls#163).

RFC 5246 6.2.3.3 requires the explicit nonce (most usually sequence
number) in the additional authenticated data (AAD), however
crypto_data.sequence, used for the AAD creation in EAEADCryptoContainer,
wasn't initialized with the explicit_nonce.
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

No branches or pull requests

2 participants