Mercurial > pidgin
comparison libpurple/protocols/oscar/tlv.c @ 30390:553cd883a29a
Removed '#if 0/endif' blocks. Some of them have the form of TODO, but
most of them are years old, so that shouldn't be a big deal.
author | ivan.komarov@soc.pidgin.im |
---|---|
date | Wed, 28 Jul 2010 19:37:26 +0000 |
parents | 0d3145e6b1d6 |
children | bbb27d65681f |
comparison
equal
deleted
inserted
replaced
30389:43dae0cda901 | 30390:553cd883a29a |
---|---|
47 aim_tlv_t *tlv; | 47 aim_tlv_t *tlv; |
48 | 48 |
49 type = byte_stream_get16(bs); | 49 type = byte_stream_get16(bs); |
50 length = byte_stream_get16(bs); | 50 length = byte_stream_get16(bs); |
51 | 51 |
52 #if 0 | |
53 /* | |
54 * This code hasn't been needed in years. It's been commented | |
55 * out since 2003, at the latest. It seems likely that it was | |
56 * just a bug in their server code that has since been fixed. | |
57 * In any case, here's the orignal comment, kept for historical | |
58 * purposes: | |
59 * | |
60 * Okay, so now AOL has decided that any TLV of | |
61 * type 0x0013 can only be two bytes, despite | |
62 * what the actual given length is. So here | |
63 * we dump any invalid TLVs of that sort. Hopefully | |
64 * there's no special cases to this special case. | |
65 * - mid (30jun2000) | |
66 */ | |
67 if ((type == 0x0013) && (length != 0x0002)) { | |
68 length = 0x0002; | |
69 return list; | |
70 } | |
71 #endif | |
72 if (length > byte_stream_empty(bs)) { | 52 if (length > byte_stream_empty(bs)) { |
73 aim_tlvlist_free(list); | 53 aim_tlvlist_free(list); |
74 return NULL; | 54 return NULL; |
75 } | 55 } |
76 | 56 |