Merge pull request #52 from cloudstruct/bugfix/set-cbor-copy

agaffney · web-flow · commit 396dee8eeda4 · 2022-03-22T20:20:56.000-05:00
Allocate buffer and copy data when setting message CBOR
diff --git a/protocol/message.go b/protocol/message.go
@@ -15,7 +15,8 @@ type MessageBase struct {
 }
 
 func (m *MessageBase) SetCbor(data []byte) {
-	m.rawCbor = data
+	m.rawCbor = make([]byte, len(data))
+	copy(m.rawCbor, data)
 }
 
 func (m *MessageBase) Cbor() []byte {

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,8 @@ type MessageBase struct {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`func (m *MessageBase) SetCbor(data []byte) {`
`18`		`- m.rawCbor = data`
	`18`	`+ m.rawCbor = make([]byte, len(data))`
	`19`	`+ copy(m.rawCbor, data)`
`19`	`20`	`}`
`20`	`21`
`21`	`22`	`func (m *MessageBase) Cbor() []byte {`