Mercurial > pidgin.yaz
comparison libpurple/protocols/qq/qq_network.c @ 24359:42261d87854a
pkt can't ever be NULL here. It is only ever allocated once with g_newa(),
which segfaults on failure.
Fixes CID #395
| author | Richard Laager <rlaager@wiktel.com> |
|---|---|
| date | Fri, 14 Nov 2008 06:23:04 +0000 |
| parents | 1ee91ff0d5fe |
| children | 75545fdf8944 |
comparison
equal
deleted
inserted
replaced
| 24358:f05c0530efe9 | 24359:42261d87854a |
|---|---|
| 459 /* purple_debug_info("TCP_PENDING", "free tcp_rxqueue\n"); */ | 459 /* purple_debug_info("TCP_PENDING", "free tcp_rxqueue\n"); */ |
| 460 g_free(conn->tcp_rxqueue); | 460 g_free(conn->tcp_rxqueue); |
| 461 conn->tcp_rxqueue = NULL; | 461 conn->tcp_rxqueue = NULL; |
| 462 } | 462 } |
| 463 | 463 |
| 464 if (pkt == NULL) { | |
| 465 continue; | |
| 466 } | |
| 467 /* packet_process may call disconnect and destory data like conn | 464 /* packet_process may call disconnect and destory data like conn |
| 468 * do not call packet_process before jump, | 465 * do not call packet_process before jump, |
| 469 * break if packet_process return FALSE */ | 466 * break if packet_process return FALSE */ |
| 470 if (packet_process(gc, pkt, pkt_len - bytes) == FALSE) { | 467 if (packet_process(gc, pkt, pkt_len - bytes) == FALSE) { |
| 471 purple_debug_info("TCP_PENDING", "Connection has been destory\n"); | 468 purple_debug_info("TCP_PENDING", "Connection has been destory\n"); |