dkimpy

bytearray index out of range in DKIM.verify()

Series trunk
Bug #1145013

Bug #1145013 reported by Stuart Gathman on 2013-03-04

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	dkimpy	Status tracked in Trunk
	Trunk	Fix Released	Medium	Stuart Gathman

Bug Description

2013Feb26 18:49:11 check_dkim: bytearray index out of range
Traceback (most recent call last):
  File "bms.py", line 1877, in check_dkim
    res = d.verify()
  File "/usr/lib/python2.6/site-packages/dkim/__init__.py", line 546, in verify
    pk = parse_public_key(base64.b64decode(pub[b'p']))
  File "/usr/lib/python2.6/site-packages/dkim/crypto.py", line 112, in parse_public_key
    x = asn1_parse(ASN1_Object, data)
  File "/usr/lib/python2.6/site-packages/dkim/asn1.py", line 57, in asn1_parse
    tag = data[i]
IndexError: bytearray index out of range

Revision history for this message

Stuart Gathman (stuart-gathman) wrote on 2013-03-04:

failing message (spam) Edit (6.5 KiB, text/plain)

Revision history for this message

Stuart Gathman (stuart-gathman) wrote on 2013-03-04:

Unfortunately, my milter does not capture the DNS record. It works now, so they must have fixed it. Now I'll have to tinker to find how to break it in a similar way. In addition to a test case and fix, the goal is to save the raw DNS record on exceptions involving the public key.

Revision history for this message

Stuart Gathman (stuart-gathman) wrote on 2013-03-14:

Committed test case and fix as revision 102.