Merge pull request #30 from Piasy/master

fix #26
ccding · Jul 26, 2018 · be486d1 · be486d1
2 parents cf47f40 + 784235e
commit be486d1
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/stun/net.go b/stun/net.go
@@ -44,7 +44,11 @@ func (c *Client) sendBindingReq(conn net.PacketConn, addr net.Addr, changeIP boo
 		attribute = newChangeReqAttribute(changeIP, changePort)
 		pkt.addAttribute(*attribute)
 	}
+	// length of fingerprint attribute must be included into crc,
+	// so we add it before calculating crc, then subtract it after calculating crc.
+	pkt.length += 8
 	attribute = newFingerprintAttribute(pkt)
+	pkt.length -= 8
 	pkt.addAttribute(*attribute)
 	// Send packet.
 	return c.send(pkt, conn, addr)