[OTR-dev] In need of a patch to libotr

[Byrd, Brendan]

This bug has been bothering me for almost a year now:

http://lists.cypherpunks.ca/pipermail/otr-users/2010-December/001891.html

I would submit a full patch, but I need somebody to review the changes, as C isn't my first language and I'm not familiar with the OTR code as a whole.  Here's the list of changes:

All of this applies to message.c / otrl_message_receiving / case OTRL_MSGTYPE_NOTOTR:


*         This case needs to try a refresh of the conversation.  Obviously, if the states are confused, leaving them in that state is NOT the best policy.  It should instead immediately start an encrypted refresh.  I think this would be a call to the "go_encrypted" sub.

*         After the refresh, the previously unencrypted side should resend the message.  OTR Messages don't get flashed by Pidgin at least, and probably others.  As such, a U->E conversation gets lost until hours later when I happen to see all of the collected messages.

Can somebody please make a patch as such?  Hell, I'll test it out as long as I can get a Windows compiled binary of it.  This

--
Brendan Byrd <byrd.b at insightcom.com<mailto:byrd.b at insightcom.com>>



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cypherpunks.ca/pipermail/otr-dev/attachments/20110830/7851aaf7/attachment.html>