diff options
| author | gdt <gdt> | 2004-12-09 14:51:03 +0000 |
|---|---|---|
| committer | gdt <gdt> | 2004-12-09 14:51:03 +0000 |
| commit | 1f4724a4dfa5ba15b592e6cad6bb274a38a2716a (patch) | |
| tree | 4c3902bf697a6e49347ab7bbc749fac710e99654 /ospfd/ospf_apiserver.c | |
| parent | 3c6dc6ab09296d03115d6bcac566b67e7feaf858 (diff) | |
| download | quagga-1f4724a4dfa5ba15b592e6cad6bb274a38a2716a.tar.bz2 quagga-1f4724a4dfa5ba15b592e6cad6bb274a38a2716a.tar.xz | |
ospf_apiserver_term:
Remove unused variable.
Add comment: this is seriously broken, and needs fixing. It appears
likely to segfault if ever called. Comments please - should I make
the fix I suggest in the comment? -gdt
Diffstat (limited to 'ospfd/ospf_apiserver.c')
| -rw-r--r-- | ospfd/ospf_apiserver.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/ospfd/ospf_apiserver.c b/ospfd/ospf_apiserver.c index 4f294f71..07b8f211 100644 --- a/ospfd/ospf_apiserver.c +++ b/ospfd/ospf_apiserver.c @@ -180,7 +180,7 @@ out: void ospf_apiserver_term (void) { - struct listnode *node, *nnode; + struct listnode *node; struct ospf_apiserver *apiserv; /* Unregister wildcard [0/0] type */ @@ -189,6 +189,11 @@ ospf_apiserver_term (void) /* 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; + * ? + */ ospf_apiserver_free (apiserv); /* Free client list itself */ |