Skip to content

Don't panic on corrupt fingerprint subpacket#66

Merged
twiss merged 1 commit intoProtonMail:masterfrom
zugzwang:master
Nov 12, 2020
Merged

Don't panic on corrupt fingerprint subpacket#66
twiss merged 1 commit intoProtonMail:masterfrom
zugzwang:master

Conversation

@zugzwang
Copy link
Contributor

The following code panics:

	data := `-----BEGIN PGP PUBLIC KEY BLOCK00000

mQ00BF00000BCAD0000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000000000000000
000000000000000000000000000000000000ABE000G0Dn000000000000000000iQ00BB0BAgAGBCG00000`
	ReadArmoredKeyRing(strings.NewReader(data))

The corrupt key was generated with go-fuzz.

@twiss twiss merged commit b4bfec6 into ProtonMail:master Nov 12, 2020
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