From 0c3588bf5e43936ea0f1e630dea500912b598c60 Mon Sep 17 00:00:00 2001 From: Martin Mares Date: Mon, 8 May 2000 14:53:22 +0000 Subject: Don't crash when the socket gets closed between updates. Also, this time not only update `remains', but update it right :) --- TODO | 2 ++ proto/bgp/packets.c | 16 +++++++++++----- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/TODO b/TODO index 7eb9bed..22e7efe 100644 --- a/TODO +++ b/TODO @@ -22,6 +22,8 @@ Core - client: paging? - client: data losses on large dumps? +- table: ocassional core dumps in rt_prune() + - bgp: timing of updates? - bgp: dump and get_route_info diff --git a/proto/bgp/packets.c b/proto/bgp/packets.c index 7a866fc..0d9b702 100644 --- a/proto/bgp/packets.c +++ b/proto/bgp/packets.c @@ -62,7 +62,7 @@ bgp_encode_prefixes(struct bgp_proto *p, byte *w, struct bgp_bucket *buck, unsig ipa_hton(a); memcpy(w, &a, bytes); w += bytes; - remains -= bytes; + remains -= bytes + 1; rem_node(&px->bucket_node); fib_delete(&p->prefix_fib, px); } @@ -232,11 +232,17 @@ bgp_fire_tx(struct bgp_conn *conn) struct bgp_proto *p = conn->bgp; unsigned int s = conn->packets_to_send; sock *sk = conn->sk; - byte *buf = sk->tbuf; - byte *pkt = buf + BGP_HEADER_LENGTH; - byte *end; + byte *buf, *pkt, *end; int type; + if (!sk) + { + conn->packets_to_send = 0; + return 0; + } + buf = sk->tbuf; + pkt = buf + BGP_HEADER_LENGTH; + if (s & (1 << PKT_SCHEDULE_CLOSE)) { bgp_close_conn(conn); @@ -284,7 +290,7 @@ bgp_schedule_packet(struct bgp_conn *conn, int type) { DBG("BGP: Scheduling packet type %d\n", type); conn->packets_to_send |= 1 << type; - if (conn->sk->tpos == conn->sk->tbuf) + if (conn->sk && conn->sk->tpos == conn->sk->tbuf) while (bgp_fire_tx(conn)) ; } -- cgit v1.2.3