From: Jaehyun Kim <jeik01.kim@samsung.com>
Date: Fri, 14 Feb 2025 08:53:59 +0000 (+0900)
Subject: Fix converting prefix to netmask
X-Git-Tag: accepted/tizen/unified/x/20250325.213550~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8951d30fe2d14de6a70ab3342a0241442ed83cb0;p=platform%2Fcore%2Fapi%2Fwifi-manager.git

Fix converting prefix to netmask

When converting a prefix to a netmask,
the existing logic could only handle inputs that were multiples of 8,
so it was modified to handle all inputs.

Change-Id: Ibdfba514dd65fd093283b1c8c09d4200dab2f59c
Signed-off-by: Jaehyun Kim <jeik01.kim@samsung.com>
---

diff --git a/src/wifi_ap.c b/src/wifi_ap.c
index 2b59ab9..f787d6c 100644
--- a/src/wifi_ap.c
+++ b/src/wifi_ap.c
@@ -164,7 +164,7 @@ int __ap_convert_netmask_to_prefix_len(const char *netmask)
 
 static in_addr_t __ap_convert_prefix_len_to_netmask(int prefix_len)
 {
-	return (prefix_len ? (in_addr_t) 0xFFFFFFFFu >> (32 - prefix_len) : 0);
+	return (prefix_len & ~31) ? 0xFFFFFFFFu : htonl(0xFFFFFFFFu << (32 - prefix_len));
 }
 
 static wifi_manager_rssi_level_e __ap_convert_dbm_to_level_24(int rssi_dbm)