diff options
| author | Paul Jakma <paul@quagga.net> | 2012-05-01 16:24:35 +0100 |
|---|---|---|
| committer | Paul Jakma <paul@quagga.net> | 2012-05-02 11:02:11 +0100 |
| commit | 7621f336e2f346edee43227f0b1ef93fe769720b (patch) | |
| tree | 2bda36681214336bf55345cd47efb2c25fc460c2 /bgpd/bgpd.c | |
| parent | 1dba254e107dd0c4254d58e9304fc6293b46fd4d (diff) | |
bgpd: Make socket init separate, so unit tests work again.
* Separate out BGP socket initialisation from bgp_get, and make it an
explicit function. Allows unit tests to work again and probably also
benefits dry-run.
* bgpd.c: (bgp_get) move socket init out...
(bgp_socket_init) to here
* bgp_main.c: and call it after dry-run.
* bgpd.h: (bgp_socket_init) add prototype
Diffstat (limited to 'bgpd/bgpd.c')
| -rw-r--r-- | bgpd/bgpd.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/bgpd/bgpd.c b/bgpd/bgpd.c index 9c8eda88..ab27783f 100644 --- a/bgpd/bgpd.c +++ b/bgpd/bgpd.c @@ -2055,13 +2055,6 @@ bgp_get (struct bgp **bgp_val, as_t *as, const char *name) bgp_router_id_set(bgp, &router_id_zebra); *bgp_val = bgp; - /* Create BGP server socket, if first instance. */ - if (list_isempty(bm->bgp)) - { - if (bgp_socket (bm->port, bm->address) < 0) - return BGP_ERR_INVALID_VALUE; - } - listnode_add (bm->bgp, bgp); return 0; @@ -5341,6 +5334,15 @@ bgp_master_init (void) } +int +bgp_socket_init (void) +{ + /* Create BGP server socket */ + if (bgp_socket (bm->port, bm->address) < 0) + return BGP_ERR_INVALID_VALUE; + return 0; +} + void bgp_init (void) { |