6671160 dladm_setzid() should have a comment explaining why it takes a link name

6673156 dladm connect-wifi failure could potentially cause core dump
6673160 icfg_get_if_list() returns an incorrect data-link list when the ICFG_INSTALLED argument is passed

2 files changed, 8 insertions, 6 deletions

diff --git a/usr/src/lib/libdladm/common/libdllink.c b/usr/src/lib/libdladm/common/libdllink.c
index 79e644c4e7..173c5435b8 100644
--- a/usr/src/lib/libdladm/common/libdllink.c
+++ b/usr/src/lib/libdladm/common/libdllink.c
@@ -252,7 +252,11 @@ dladm_linkduplex2str(link_duplex_t duplex, char *buf)
 }
 
 /*
- * Set zoneid of a given link
+ * Set zoneid of a given link. Note that this function takes a link name
+ * argument instead of a linkid, because a data-link (and its linkid) could
+ * be created implicitly as the result of this function. For example, a VLAN
+ * could be created if a VLAN PPA hack name is assigned to an exclusive
+ * non-global zone.
  */
 dladm_status_t
 dladm_setzid(const char *link, zoneid_t zoneid)
diff --git a/usr/src/lib/libdladm/common/libdlwlan.c b/usr/src/lib/libdladm/common/libdlwlan.c
index 9a5bc6bcd0..00d78ab79b 100644
--- a/usr/src/lib/libdladm/common/libdlwlan.c
+++ b/usr/src/lib/libdladm/common/libdlwlan.c
@@ -530,12 +530,10 @@ dladm_wlan_connect(datalink_id_t linkid, dladm_wlan_attr_t *attrp,
 	dladm_status_t		status;
 
 	if ((status = dladm_wlan_validate(linkid)) != DLADM_STATUS_OK)
-		goto done;
+		return (status);
 
-	if ((gbuf = malloc(MAX_BUF_LEN)) == NULL) {
-		status = DLADM_STATUS_NOMEM;
-		goto done;
-	}
+	if ((gbuf = malloc(MAX_BUF_LEN)) == NULL)
+		return (DLADM_STATUS_NOMEM);
 
 	if ((status = do_get_linkstatus(linkid, gbuf)) != DLADM_STATUS_OK)
 		goto done;