diff options
author | hasso <hasso> | 2005-09-16 14:44:23 +0000 |
---|---|---|
committer | hasso <hasso> | 2005-09-16 14:44:23 +0000 |
commit | a96d8d1095da09811e343a534f5801a948ae1061 (patch) | |
tree | 572fdf17817de64a85cda87d15f3b0da8640c6b5 | |
parent | 53a6f932032a30625358414ae0f77790656ccaca (diff) |
* isis_lsp.c (lsp_update): Remove LSP from database before updating
its data and put it back after. Database entry MUST contain at least
correct pointers to the sysid to get correct compare results.
* isis_lsp.[ch], isis_pdu.c: Pass level to the lsp_update() function.
-rw-r--r-- | isisd/ChangeLog | 7 | ||||
-rw-r--r-- | isisd/isis_lsp.c | 11 | ||||
-rw-r--r-- | isisd/isis_lsp.h | 2 | ||||
-rw-r--r-- | isisd/isis_pdu.c | 5 |
4 files changed, 21 insertions, 4 deletions
diff --git a/isisd/ChangeLog b/isisd/ChangeLog index 822d856b..6a2be940 100644 --- a/isisd/ChangeLog +++ b/isisd/ChangeLog @@ -1,3 +1,10 @@ +2005-09-16 Hasso Tepper <hasso at quagga.net> + + * isis_lsp.c (lsp_update): Remove LSP from database before updating + its data and put it back after. Database entry MUST contain at least + correct pointers to the sysid to get correct compare results. + * isis_lsp.[ch], isis_pdu.c: Pass level to the lsp_update() function. + 2005-09-05 Hasso Tepper <hasso at quagga.net> * *.c: Try to be less verbose by default (without any debug options diff --git a/isisd/isis_lsp.c b/isisd/isis_lsp.c index 2a9d1465..d147c650 100644 --- a/isisd/isis_lsp.c +++ b/isisd/isis_lsp.c @@ -416,8 +416,14 @@ lsp_update_data (struct isis_lsp *lsp, struct stream *stream, void lsp_update (struct isis_lsp *lsp, struct isis_link_state_hdr *lsp_hdr, - struct stream *stream, struct isis_area *area) + struct stream *stream, struct isis_area *area, int level) { + dnode_t *dnode; + + /* Remove old LSP from LSP database. */ + dnode = dict_lookup (area->lspdb[level], lsp->lsp_header->lsp_id); + dnode_destroy (dict_delete (area->lspdb[level], dnode)); + /* free the old lsp data */ XFREE (MTYPE_STREAM_DATA, lsp->pdu); lsp_clear_data (lsp); @@ -427,6 +433,9 @@ lsp_update (struct isis_lsp *lsp, struct isis_link_state_hdr *lsp_hdr, /* set the new values for lsp header */ memcpy (lsp->lsp_header, lsp_hdr, ISIS_LSP_HDR_LEN); + + /* Put LSP back into LSP database, now with updated data. */ + lsp_insert (lsp, area->lspdb[level]); } /* creation of LSP directly from what we received */ diff --git a/isisd/isis_lsp.h b/isisd/isis_lsp.h index 8e450668..fb6f1d82 100644 --- a/isisd/isis_lsp.h +++ b/isisd/isis_lsp.h @@ -114,7 +114,7 @@ int lsp_id_cmp (u_char * id1, u_char * id2); int lsp_compare (char *areatag, struct isis_lsp *lsp, u_int32_t seq_num, u_int16_t checksum, u_int16_t rem_lifetime); void lsp_update (struct isis_lsp *lsp, struct isis_link_state_hdr *lsp_hdr, - struct stream *stream, struct isis_area *area); + struct stream *stream, struct isis_area *area, int level); void lsp_inc_seqnum (struct isis_lsp *lsp, u_int32_t seq_num); int lsp_print_all (struct vty *vty, dict_t * lspdb, char detail, char dynhost); diff --git a/isisd/isis_pdu.c b/isisd/isis_pdu.c index b15d4551..87f37901 100644 --- a/isisd/isis_pdu.c +++ b/isisd/isis_pdu.c @@ -1085,7 +1085,8 @@ dontcheckadj: /* 7.3.16.4 b) 1) */ if (comp == LSP_NEWER) { - lsp_update (lsp, hdr, circuit->rcv_stream, circuit->area); + lsp_update (lsp, hdr, circuit->rcv_stream, circuit->area, + level); /* ii */ ISIS_FLAGS_SET_ALL (lsp->SRMflags); /* iii */ @@ -1239,7 +1240,7 @@ dontcheckadj: else if (comp == LSP_EQUAL) { ISIS_CLEAR_FLAG (lsp->SRMflags, circuit); - lsp_update (lsp, hdr, circuit->rcv_stream, circuit->area); + lsp_update (lsp, hdr, circuit->rcv_stream, circuit->area, level); if (circuit->circ_type != CIRCUIT_T_BROADCAST) { ISIS_SET_FLAG (lsp->SSNflags, circuit); |