You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@plc4x.apache.org by sr...@apache.org on 2022/08/04 12:37:34 UTC

[plc4x] 01/03: fix(plc4xpcapanalyzer): fixed unnecessary replacing of payloads

This is an automated email from the ASF dual-hosted git repository.

sruehl pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/plc4x.git

commit 4d379b194b8321700ea403ef02da4546ae1ab740
Author: Sebastian Rühl <sr...@apache.org>
AuthorDate: Thu Aug 4 11:38:28 2022 +0200

    fix(plc4xpcapanalyzer): fixed unnecessary replacing of payloads
---
 plc4go/tools/plc4xpcapanalyzer/internal/cbusanalyzer/analyzer.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plc4go/tools/plc4xpcapanalyzer/internal/cbusanalyzer/analyzer.go b/plc4go/tools/plc4xpcapanalyzer/internal/cbusanalyzer/analyzer.go
index 796228f02..f6573b92f 100644
--- a/plc4go/tools/plc4xpcapanalyzer/internal/cbusanalyzer/analyzer.go
+++ b/plc4go/tools/plc4xpcapanalyzer/internal/cbusanalyzer/analyzer.go
@@ -283,7 +283,7 @@ func (a *Analyzer) MapPackets(in chan gopacket.Packet, packetInformationCreator
 					} else {
 						currentApplicationLayer := packet.ApplicationLayer()
 						newPayload := gopacket.Payload(payload)
-						if !reflect.DeepEqual(currentApplicationLayer.Payload(), newPayload) {
+						if !reflect.DeepEqual(currentApplicationLayer.Payload(), payload) {
 							log.Debug().Msgf("Replacing payload %q with %q", currentApplicationLayer.Payload(), payload)
 							packet = &manipulatedPackage{Packet: packet, newApplicationLayer: newPayload}
 						}