Suggested changes to PurpleCipherOps and need for code review

Ethan Blanton elb at pidgin.im
Sun May 5 19:32:51 EDT 2013


Tomasz Wasilczyk spake unto us the following wisdom:
> I did all changes I posted here, but I would like to do one more.
> 
> Currently, purple_cipher_context_reset is hardly usable, because it
> resets both state (IV, digest context) and configuration (key, ops).
> When using simple ciphers, like md5, it's not a problem, but for hmac
> it is - we need to set key and hash type every time, when we want to
> digest a value. I suggest doing a void, but significant change in API:

[snip reset that only resets some values]

I think this should probably be a separate function.  They serve
different purposes.

Ethan




More information about the Devel mailing list