<html><body>

<p>libical/src/libical/ (icalattach.c icalattach.h): <br />13:25  CIA-14: libical: * unsigned char* why?<br />13:25  CIA-14: libical: * remove const casts; this is a place where casting away const definitely gave us a bug, as fixed by Allen earlier.</p>
<p>Allen, since you've fixed a crash in R903 and the experiences we've collected with libicals code so far...</p>
<p>Its probably always a good idea to question the surrounding code and have a look the place where they're called.</p>
<p>The caller of icalattach_new_from_data() casted away a const char* which most probably was the reason for this crash?</p>
<p>so, having a look at the bigger picture around a bug often shows more places for code cleanup, or even possible removal for now abandoned code;</p>
<p>maybe somewhen in the future libical will have clean code ;-)</p>
</body></html>