From fb0093754d08b2e3d98b28b4ef56e277a82f100e Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Tue, 15 Jun 2004 15:19:23 +0000 Subject: Use camel_multipart_signed_get_content_stream() rather than getting the 2004-06-15 Jeffrey Stedfast * camel-gpg-context.c (gpg_verify): Use camel_multipart_signed_get_content_stream() rather than getting the first part and canonicalising it ourselves. Fixes bug #60159. svn path=/trunk/; revision=26350 --- camel/ChangeLog | 4 ++++ camel/camel-gpg-context.c | 23 ++++++++++++----------- 2 files changed, 16 insertions(+), 11 deletions(-) (limited to 'camel') diff --git a/camel/ChangeLog b/camel/ChangeLog index 3abe96ac18..1001ff425b 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,5 +1,9 @@ 2004-06-15 Jeffrey Stedfast + * camel-gpg-context.c (gpg_verify): Use + camel_multipart_signed_get_content_stream() rather than getting + the first part and canonicalising it ourselves. Fixes bug #60159. + * providers/imap4/camel-imap4-stream.c (camel_imap4_stream_init): Init have_unget to FALSE. Don't set unget to NULL, it's no longer a pointer. diff --git a/camel/camel-gpg-context.c b/camel/camel-gpg-context.c index adcd40cb17..7f7a493bce 100644 --- a/camel/camel-gpg-context.c +++ b/camel/camel-gpg-context.c @@ -1385,7 +1385,7 @@ gpg_verify (CamelCipherContext *context, CamelMimePart *ipart, CamelException *e struct _GpgCtx *gpg = NULL; char *sigfile = NULL; CamelContentType *ct; - CamelMimePart *sigpart, *datapart; + CamelMimePart *sigpart; CamelStream *istream = NULL; CamelMultipart *mps; @@ -1401,15 +1401,19 @@ gpg_verify (CamelCipherContext *context, CamelMimePart *ipart, CamelException *e return NULL; } - datapart = camel_multipart_get_part(mps, CAMEL_MULTIPART_SIGNED_CONTENT); - sigpart = camel_multipart_get_part(mps, CAMEL_MULTIPART_SIGNED_SIGNATURE); - - if (sigpart == NULL || datapart == NULL) { + if (!(istream = camel_multipart_signed_get_content_stream ((CamelMultipartSigned *) mps, NULL))) { camel_exception_setv (ex, CAMEL_EXCEPTION_SYSTEM, _("Cannot verify message signature: Incorrect message format")); - goto exception; + return NULL; } - + + if (!(sigpart = camel_multipart_get_part (mps, CAMEL_MULTIPART_SIGNED_SIGNATURE))) { + camel_exception_setv (ex, CAMEL_EXCEPTION_SYSTEM, + _("Cannot verify message signature: Incorrect message format")); + camel_object_unref (istream); + return NULL; + } + sigfile = swrite (sigpart); if (sigfile == NULL) { camel_exception_setv (ex, CAMEL_EXCEPTION_SYSTEM, @@ -1417,11 +1421,8 @@ gpg_verify (CamelCipherContext *context, CamelMimePart *ipart, CamelException *e g_strerror (errno)); goto exception; } - - istream = camel_stream_mem_new(); - camel_cipher_canonical_to_stream(datapart, CAMEL_MIME_FILTER_CANON_CRLF, istream); - camel_stream_reset(istream); + camel_stream_reset(istream); gpg = gpg_ctx_new (context->session); gpg_ctx_set_mode (gpg, GPG_CTX_MODE_VERIFY); gpg_ctx_set_hash (gpg, camel_cipher_id_to_hash(context, camel_content_type_param(ct, "micalg"))); -- cgit v1.2.3