On Mon, 2012-01-09 at 19:42 -0500, Ari Pollak wrote:
> On 01/09/2012 04:06 PM, Adam D. Barratt wrote:
> > Something odd appears to have happened to CVE-2011-4602.patch; it looks
> > like a diff of a brokenly wrapped diff.  For instance: (sic)
> 
> Strange, but it actually doesn't affect the patch, and the actual 
> changes are correct. I'll remove the extra "comments" from that patch.

Thanks.  Would it be possible to have an updated diff without the cruft?

(fwiw, I'm also not quite sure what some other parts of that patch have
to do with fixing the crash; e.g.

+-      is_audio = !strcmp(media_type, "audio");
++      is_audio = g_str_equal(media_type, "audio");
[...]
+-              type = is_audio == TRUE ? PURPLE_MEDIA_RECV_AUDIO
++              type = is_audio ? PURPLE_MEDIA_RECV_AUDIO
)

Regards,

Adam




-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/1326558826.29770.51.ca...@jacala.jungle.funky-badger.org

Reply via email to