Re: Double-free on message signature verify in silc-client - Patch included
Pekka Riikonen <[email protected]> Tue, 25 May 2010 06:27:17 +0200 (CEST)
| Newsgroups | gmane.network.silc.devel |
|---|---|
| Message-ID | <[email protected]> |
: If the calculated fingerprint of the public key is the same as the : fingerprint sent by the server then everything is fine : (strcmp(fingerprint, fingerprint2) returns 0). But if those fingerprints : differ, the double-free takes place and the client reliably crashes. : : My proposed solution (found in the attached patch with the funny name) : is to simply leave the function with SILC_MSG_SIGNED_FAILED instead of : executing the rest of the function (which is exactly what that existing : comment says it should do). : Thanks for reporting. I fixed it the same way except I return UNKNOWN instead of FAILED. We don't want to do the verification in that case at all and the signature status is UNKNOWN. Pekka ________________________________________________________________________ Pekka Riikonen priikone at silcnet.org Secure Internet Live Conferencing (SILC) http://silcnet.org/ _______________________________________________________________________ Info: https://lists.silcnet.org/mailman/listinfo/silc-announce Archive: https://lists.silcnet.org/pipermail/silc-announce FAQ: http://silcnet.org/support/faq/