[Evolution-hackers] Null check for em-format-part

[Vivek Jain <jvivek novell com>]

hey!

 I have a question regarding function em_format_part(). I observe that
when "part" is NULL, it eventually crashes evolution, as we end up
accessing members of a structure pointer having NULL value.

My doubt is, in which of the cases it is safe to assume that, we will
surely get the "part", and we can have a call like.

em_format_part(emf, stream, camel_multipart_get_part(mp, 1));

I am wondering if we should NULL check "part" every time before using
it?

Thanks,
Vivek 

PS: This is with reference to the bug: 
http://bugzilla.gnome.org/show_bug.cgi?id=304013