diff options
| author | gdt <gdt> | 2004-12-09 15:51:03 +0100 |
|---|---|---|
| committer | gdt <gdt> | 2004-12-09 15:51:03 +0100 |
| commit | 91f3e525a95d6a86f916872116af162eb3bdae5d (patch) | |
| tree | c9944157d5cde3acf11850420012814214f8488c /ospfd/ospf_apiserver.c | |
| parent | add comments about warnings (diff) | |
| download | frr-91f3e525a95d6a86f916872116af162eb3bdae5d.tar.xz frr-91f3e525a95d6a86f916872116af162eb3bdae5d.zip | |
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 4f294f718..07b8f211a 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 */ |