diff options
| author | gdt <gdt> | 2004-12-09 16:30:04 +0000 |
|---|---|---|
| committer | gdt <gdt> | 2004-12-09 16:30:04 +0000 |
| commit | f60528ecef682f72c4e0552f5f4dd0bc811ea02a (patch) | |
| tree | a36324dd3402a9f10bc0bbf6dafb7a12e7134c33 /ospfd/ospf_apiserver.c | |
| parent | 57d86be6f9e38768984bfbe8cb6ae0978d96a6e4 (diff) | |
| download | quagga-f60528ecef682f72c4e0552f5f4dd0bc811ea02a.tar.bz2 quagga-f60528ecef682f72c4e0552f5f4dd0bc811ea02a.tar.xz | |
* ospf_apiserver.c (ospf_apiserver_term): Obtain struct
ospf_apiserver * from listnode. Remove unused variables. Follows
suggestion from Jay Fenlason.
Diffstat (limited to 'ospfd/ospf_apiserver.c')
| -rw-r--r-- | ospfd/ospf_apiserver.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/ospfd/ospf_apiserver.c b/ospfd/ospf_apiserver.c index 07b8f211..89af2a36 100644 --- a/ospfd/ospf_apiserver.c +++ b/ospfd/ospf_apiserver.c @@ -180,20 +180,17 @@ out: void ospf_apiserver_term (void) { - struct listnode *node; struct ospf_apiserver *apiserv; /* Unregister wildcard [0/0] type */ ospf_delete_opaque_functab (0 /* all LSAs */, 0 /* all opaque types */); - /* Free all client instances */ - while ( (node = listhead (apiserver_list)) != NULL) - /* - * XXX: this is just plain odd/wrong. Is there a missing - * apiserv = (struct ospf_apiserver *) node; - * ? - */ + /* + * Free all client instances. ospf_apiserver_free removes the node + * from the list, so we examine the head of the list anew each time. + */ + while ( (apiserv = getdata (listhead (apiserver_list))) != NULL) ospf_apiserver_free (apiserv); /* Free client list itself */ |