summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMitesh Kanjariya <mitesh@cumulusnetworks.com>2017-08-25 16:36:31 -0700
committerMitesh Kanjariya <mitesh@cumulusnetworks.com>2017-08-25 16:36:31 -0700
commit0802e118dd634b37d580effb7e5df4d4c26fa35c (patch)
tree0045e3e79d0b7401ec01d9790b18542fd75c1335
parent1f53ef552de070d22a69558baafca8f8c5b9eecd (diff)
Handle hostname/domainname properly for FreeBSD
Signed-off-by: Mitesh Kanjariya <mitesh@cumulusnetworks.com>
-rw-r--r--bgpd/bgpd.c10
-rw-r--r--lib/command.c14
2 files changed, 6 insertions, 18 deletions
diff --git a/bgpd/bgpd.c b/bgpd/bgpd.c
index c64ebf1222..8b70c24ac2 100644
--- a/bgpd/bgpd.c
+++ b/bgpd/bgpd.c
@@ -2766,15 +2766,17 @@ static struct bgp *bgp_create(as_t *as, const char *name,
XFREE(MTYPE_BGP_PEER_HOST, bgp->peer_self->hostname);
bgp->peer_self->hostname = NULL;
}
- bgp->peer_self->hostname = XSTRDUP(MTYPE_BGP_PEER_HOST,
- hostname_get());
+ if (hostname_get())
+ bgp->peer_self->hostname = XSTRDUP(MTYPE_BGP_PEER_HOST,
+ hostname_get());
if (bgp->peer_self->domainname != NULL) {
XFREE(MTYPE_BGP_PEER_HOST, bgp->peer_self->domainname);
bgp->peer_self->domainname = NULL;
}
- bgp->peer_self->domainname = XSTRDUP(MTYPE_BGP_PEER_HOST,
- domainname_get());
+ if (domainname_get())
+ bgp->peer_self->domainname = XSTRDUP(MTYPE_BGP_PEER_HOST,
+ domainname_get());
bgp->peer = list_new();
bgp->peer->cmp = (int (*)(void *, void *))peer_cmp;
bgp->peerhash = hash_create(peer_hash_key_make, peer_hash_same, NULL);
diff --git a/lib/command.c b/lib/command.c
index 78d8dbdc62..2d262c1146 100644
--- a/lib/command.c
+++ b/lib/command.c
@@ -130,12 +130,6 @@ struct host host;
*/
const char *hostname_get(void)
{
- struct utsname names;
-
- if (!host.name) {
- uname(&names);
- host.name = XSTRDUP(MTYPE_HOST, names.nodename);
- }
return host.name;
}
@@ -144,14 +138,6 @@ const char *hostname_get(void)
*/
const char *domainname_get(void)
{
- struct utsname names;
-
- if (!host.name || !host.domainname) {
-#ifdef HAVE_STRUCT_UTSNAME_DOMAINNAME
- uname(&names);
- host.domainname = XSTRDUP(MTYPE_HOST, names.domainname);
-#endif
- }
return host.domainname;
}