Skip to content

Commit 9e8f869

Browse files
During signature validation, do not store any state until entire validation
is done. Signed-off-by: Fredrik Skogman <kommendorkapten@github.com>
1 parent 14d83fb commit 9e8f869

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

pkg/types/cose/v0.0.1/entry.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -239,15 +239,16 @@ func (v *V001Entry) validate() error {
239239
if err != nil {
240240
return err
241241
}
242-
v.sign1Msg = gocose.NewSign1Message()
243-
if err := v.sign1Msg.UnmarshalCBOR(v.CoseObj.Message); err != nil {
242+
sign1Msg := gocose.NewSign1Message()
243+
if err := sign1Msg.UnmarshalCBOR(v.CoseObj.Message); err != nil {
244244
return err
245245
}
246246

247-
if err := v.sign1Msg.Verify(v.CoseObj.Data.Aad, bv); err != nil {
247+
if err := sign1Msg.Verify(v.CoseObj.Data.Aad, bv); err != nil {
248248
return err
249249
}
250250

251+
v.sign1Msg = sign1Msg
251252
return nil
252253
}
253254

0 commit comments

Comments
 (0)