Re: [evolution-patches] Patch for Bugzilla bugs 72285 and 71845

[Not Zed <notzed ximian com>]

Its hitting an assertion - an assertion is only there for a reason.  It is invalid to have an address with no address part.  It seems from the xml reponse you want to set an rfc822 'group' address, but camel-internet-address doesn't support group addressing.

You probably need to do something like this:
 - use camel_header_address_* to form the address structure
 - use camel_header_address_list_encode to turn it into a properly rfc822 address string
 - use camel_medium_set_header(msg, "From", string) to set the 'from' line on the address for that message

You probably need to do the same with all the other address fields as well.  It shouldn't be much different to the code you already have.

The other alternative is make camel_internet_address support groups, but that would be a pretty major/far reaching change, whch seems more effort than its worth, which is why it hasn't been done yet.


On Thu, 2005-02-17 at 16:11 +0530, Sivaiah Nallagatla wrote:
> 71845 is unrelated to 72285 and will not be fixed by this patch.
> This patch solve the 72285 crash,  butter it is better to get opinion
> from Micheal or Jeff about having empty string as email 
>
> Siva
>
> On Thu, 2005-02-17 at 15:15 +0530, Sankar P wrote:
> >Hi,
> >
> >When a mail containing a null From address field is Opened or Moved for
> >Offline View, evolution crashes. 
> >
> >The attached Patch will fix this and hence solves bugs 72285 and 71845.
> >
> >Thanks & Regards,
> >Sankar P