reduce part of sendWorker that needs to keep a mutex

src/yggdrasil/session.go

@@ -492,23 +492,27 @@ func (sinfo *sessionInfo) sendWorker() {
 		case <-sinfo.cancel.Finished():
 			return
 		case bs := <-sinfo.send:
-			// TODO
+			var p wire_trafficPacket
-			var packet []byte
+			var k crypto.BoxSharedKey
 			sessionFunc := func() {
-				defer util.PutBytes(bs)
 				sinfo.bytesSent += uint64(len(bs))
-				payload, nonce := crypto.BoxSeal(&sinfo.sharedSesKey, bs, &sinfo.myNonce)
+				p = wire_trafficPacket{
-				defer util.PutBytes(payload)
+					Coords: append([]byte(nil), sinfo.coords...),
-				// Construct the wire packet to send to the router
-				p := wire_trafficPacket{
-					Coords:  sinfo.coords,
 					Handle: sinfo.theirHandle,
-					Nonce:   *nonce,
+					Nonce:  sinfo.myNonce,
-					Payload: payload,
 				}
-				packet = p.encode()
+				sinfo.myNonce.Increment()
+				k = sinfo.sharedSesKey
 			}
+			// Get the mutex-protected info needed to encrypt the packet
 			sinfo.doFunc(sessionFunc)
+			// Encrypt the packet
+			p.Payload, _ = crypto.BoxSeal(&k, bs, &p.Nonce)
+			packet := p.encode()
+			// Cleanup
+			util.PutBytes(bs)
+			util.PutBytes(p.Payload)
+			// Send the packet
 			sinfo.core.router.out(packet)
 		}
 	}