community
/
dns
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

unpack: allow for multiple edns0 options

This commit is contained in:
Miek Gieben 2012-12-09 21:11:15 +01:00
parent 66df243b0e
commit 29fa0ac7c0
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
msg.go
View File

@ -711,14 +711,14 @@ func unpackStructValue(val reflect.Value, msg []byte, off int) (off1 int, err er
}
fv.Set(reflect.ValueOf(txt))
case `dns:"opt"`: // edns0
// TODO: multiple EDNS0 options
rdlength := int(val.FieldByName("Hdr").FieldByName("Rdlength").Uint())
if rdlength == 0 {
// This is an EDNS0 (OPT Record) with no rdata
// We can savely return here.
// We can safely return here.
break
}
edns := make([]EDNS0, 0)
Option:
code := uint16(0)
if off+2 > lenmsg {
return lenmsg, &Error{Err: "overflow unpacking opt"}
@ -745,8 +745,10 @@ func unpackStructValue(val reflect.Value, msg []byte, off int) (off1 int, err er
edns = append(edns, e)
off = off1 + int(optlen)
}
if off < rdlength {
goto Option
}
fv.Set(reflect.ValueOf(edns))
// multiple EDNS codes?
case `dns:"a"`:
if off+net.IPv4len > lenmsg {
return lenmsg, &Error{Err: "overflow unpacking a"}