[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#655178: pu: package pidgin/2.7.3-1+squeeze1



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




Reply to: