Skip to content

fcl-base: blowfish2: Remove unnecessary destructor from TBlowFish2Stream

Summary

The only operation in the TBlowFish2Stream destructor is calling FreeAndNil for a property, which type is not a TObject descendant but a record. That leads to an Access Violation. FreeAndNil call is not needed, therefore the whole destructor can be removed.

What is the current bug behavior?

Calling a destructor on either encrypting or decrypting stream always results in an Access Violation.

What is the behavior after applying this patch?

Blowfish2 streams can be disposed properly.

Edited by Oleg Latov

Merge request reports

Loading