BNetworkAddress: Address review comments.

Thanks Axel and Rene!
haiku · Jul 28, 2015 · d682f9f · d682f9f
1 parent 71b756a
commit d682f9f
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/src/kits/network/libnetapi/NetworkAddress.cpp b/src/kits/network/libnetapi/NetworkAddress.cpp
@@ -151,7 +151,7 @@ BNetworkAddress::Unset()
 {
 	fAddress.ss_family = AF_UNSPEC;
 	fAddress.ss_len = 2;
-	fStatus = B_OK;
+	fStatus = B_NO_INIT;
 }
 
 
@@ -177,8 +177,10 @@ BNetworkAddress::SetTo(const char* host, uint16 port, uint32 flags)
 
 	cookie = 0;
 	status = resolver->GetNextAddress(&cookie, *this);
-	if (status == B_OK)
-		fStatus = B_OK;
+	if (status != B_OK) {
+		Unset();
+		fStatus = status;
+	}
 	return status;
 }
 
@@ -205,8 +207,10 @@ BNetworkAddress::SetTo(const char* host, const char* service, uint32 flags)
 
 	cookie = 0;
 	status = resolver->GetNextAddress(&cookie, *this);
-	if (status == B_OK)
-		fStatus = B_OK;
+	if (status != B_OK) {
+		Unset();
+		fStatus = status;
+	}
 	return status;
 }
 
@@ -231,8 +235,10 @@ BNetworkAddress::SetTo(int family, const char* host, uint16 port, uint32 flags)
 
 	uint32 cookie = 0;
 	status = resolver->GetNextAddress(&cookie, *this);
-	if (status == B_OK)
-		fStatus = B_OK;
+	if (status != B_OK) {
+		Unset();
+		fStatus = status;
+	}
 	return status;
 }
 
@@ -258,8 +264,10 @@ BNetworkAddress::SetTo(int family, const char* host, const char* service,
 
 	uint32 cookie = 0;
 	status = resolver->GetNextAddress(&cookie, *this);
-	if (status == B_OK)
-		fStatus = B_OK;
+	if (status != B_OK) {
+		Unset();
+		fStatus = status;
+	}
 	return status;
 }