Snclient off by one nrpe (#368)

Author: egonrassi

pkg/nrpe/nrpe.go

@@ -109,8 +109,8 @@ func BuildPacketV2(packetType, statusCode uint16, statusLine []byte) *Packet {
 func BuildPacketV4(packetType, statusCode uint16, statusLine []byte) *Packet {
 	dataLength := len(statusLine) + 1 // +1 for the final null byte
 	if dataLength > NrpeV4MaxPacketDataLength {
-		statusLine = statusLine[0:NrpeV4MaxPacketDataLength]
-		dataLength = len(statusLine)
+		statusLine = statusLine[0 : NrpeV4MaxPacketDataLength-1]
+		dataLength = NrpeV4MaxPacketDataLength
 	}
 	if dataLength < NrpeV2MaxPacketDataLength {
 		dataLength = NrpeV2MaxPacketDataLength

pkg/snclient/check_swap_io.go

@@ -159,7 +159,7 @@ func (l *CheckSwapIO) addSwapRate(check *CheckData, snc *Agent) {
 	} else {
 		message := "swapOutCounter does not have a last value"
 		log.Debug(message)
-		entry["_error "] = entry["_error"] + " | " + message
+		entry["_error"] = entry["_error"] + " | " + message
 	}
 
 	if err := swapOutCounter.CheckRetention(time.Duration(l.lookback)*time.Second, 0); err != nil {