Fix: mac address overflow on windows (#3983)

* Fix: mac address overflow on windows PIP_ADAPTER_ADDRESSES->PhysicalAddressLength will get max up to 8 bytes, but we have only 6 bytes for mac address. In some case, GetAdaptersAddresses() will report address with PhysicalAddressLength=8, e.g. "Microsoft Teredo Tunneling Adapter", which will cause crash at DomainParticipantFactory::create_participant() Signed-off-by: [email protected] <[email protected]> Signed-off-by: Fei Chong <[email protected]> * Fix build error due to windows `min` macro Signed-off-by: Miguel Company <[email protected]> --------- Signed-off-by: [email protected] <[email protected]> Signed-off-by: Fei Chong <[email protected]> Signed-off-by: Miguel Company <[email protected]> Co-authored-by: Miguel Company <[email protected]> (cherry picked from commit 2601e95)
eProsima · Nov 14, 2023 · 55a29d9 · 55a29d9
1 parent 8d0b35a
commit 55a29d9
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/cpp/utils/IPFinder.cpp b/src/cpp/utils/IPFinder.cpp
@@ -265,7 +265,7 @@ bool IPFinder::getAllMACAddress(
         if (aa->OperStatus == 1) //is ENABLED
         {
             info_MAC mac;
-            memcpy(mac.address, aa->PhysicalAddress, aa->PhysicalAddressLength);
+            memcpy(mac.address, aa->PhysicalAddress, (std::min)(6UL, aa->PhysicalAddressLength));
 
             if (std::find(macs->begin(), macs->end(), mac) == macs->end())
             {