LudovicRousseau / PyKCS11 / 12965932525

94%

Build Type

push

github

Committed by LudovicRousseau

Commit Message

bugfix: store CKM_CONCATENATE_BASE_AND_KEY parameter in mechanism context

It's needed to be sure that encKey will not be destroyed by the garbage
collector while the mechanism object is still in use. I guess this is
necessary because the assignment:

self._mech.pParameter = self._encKey

internally copies only a pointer to self._encKey, which would get
invalidated if one destroys self._encKey and that's why it should be
attached to self. It wasn't obvious for me either, but this is how it
works (I guess). The other mechanisms do pretty much the same thing --
i.e. they store the parameter in self._param usually.

Note:
.pParameter is just a property of a CK_MECHANISM() object.

Run Details

0 of 7 new or added lines in 2 files covered. (0.0%)

1 existing line in 1 file now uncovered.

2993 of 3457 relevant lines covered (86.58%)

0.87 hits per line