--- a/src/wifi/model/mgt-headers.cc Wed May 15 14:45:28 2013 -0400
+++ b/src/wifi/model/mgt-headers.cc Wed May 15 15:17:20 2013 -0400
@@ -484,42 +484,46 @@
{
case BLOCK_ACK_ADDBA_REQUEST:
retval.blockAck = BLOCK_ACK_ADDBA_REQUEST;
- return retval;
+ break ;
case BLOCK_ACK_ADDBA_RESPONSE:
retval.blockAck = BLOCK_ACK_ADDBA_RESPONSE;
- return retval;
+ break ;
case BLOCK_ACK_DELBA:
retval.blockAck = BLOCK_ACK_DELBA;
- return retval;
+ break ;
}
+ break ;
+
case MESH_PEERING_MGT:
switch (m_actionValue)
{
case PEER_LINK_OPEN:
retval.peerLink = PEER_LINK_OPEN;
- return retval;
+ break ;
case PEER_LINK_CONFIRM:
retval.peerLink = PEER_LINK_CONFIRM;
- return retval;
+ break ;
case PEER_LINK_CLOSE:
retval.peerLink = PEER_LINK_CLOSE;
- return retval;
+ break ;
default:
NS_FATAL_ERROR ("Unknown mesh peering management action code");
retval.peerLink = PEER_LINK_OPEN; /* quiet compiler */
- return retval;
}
+ break ;
+
case MESH_PATH_SELECTION:
switch (m_actionValue)
{
case PATH_SELECTION:
retval.pathSelection = PATH_SELECTION;
- return retval;
+ break ;
default:
NS_FATAL_ERROR ("Unknown mesh path selection action code");
retval.peerLink = PEER_LINK_OPEN; /* quiet compiler */
- return retval;
}
+ break ;
+
case MESH_LINK_METRIC:
// not yet supported
case MESH_INTERWORKING:
@@ -529,8 +533,8 @@
default:
NS_FATAL_ERROR ("Unsupported mesh action");
retval.peerLink = PEER_LINK_OPEN; /* quiet compiler */
- return retval;
}
+ return retval;
}
TypeId
WifiActionHeader::GetTypeId ()
--- a/src/wifi/model/regular-wifi-mac.cc Wed May 15 14:45:28 2013 -0400
+++ b/src/wifi/model/regular-wifi-mac.cc Wed May 15 15:17:20 2013 -0400
@@ -512,10 +512,13 @@
default:
NS_FATAL_ERROR ("Unsupported Action field in Block Ack Action frame");
+ return;
}
+
default:
NS_FATAL_ERROR ("Unsupported Action frame received");
+ return;
}
}
NS_FATAL_ERROR ("Don't know how to handle frame (type=" << hdr->GetType ());
--- a/src/wifi/model/wifi-mac-header.cc Wed May 15 14:45:28 2013 -0400
+++ b/src/wifi/model/wifi-mac-header.cc Wed May 15 15:17:20 2013 -0400
@@ -221,12 +221,15 @@
case WIFI_MAC_MGT_DEAUTHENTICATION:
m_ctrlType = TYPE_MGT;
m_ctrlSubtype = 12;
+ break;
case WIFI_MAC_MGT_ACTION:
m_ctrlType = TYPE_MGT;
m_ctrlSubtype = 13;
+ break;
case WIFI_MAC_MGT_ACTION_NO_ACK:
m_ctrlType = TYPE_MGT;
m_ctrlSubtype = 14;
+ break;
case WIFI_MAC_MGT_MULTIHOP_ACTION:
m_ctrlType = TYPE_MGT;
m_ctrlSubtype = 15;
@@ -999,10 +1002,12 @@
os << " Duration/ID=" << m_duration << "us"
<< "DA=" << m_addr1 << ", SA=" << m_addr2 << ", BSSID=" << m_addr3
<< ", FragNumber=" << std::hex << (int) m_seqFrag << std::dec << ", SeqNumber=" << m_seqSeq;
+ break;
case WIFI_MAC_MGT_MULTIHOP_ACTION:
os << " Duration/ID=" << m_duration << "us"
<< "RA=" << m_addr1 << ", TA=" << m_addr2 << ", DA=" << m_addr3
<< ", FragNumber=" << std::hex << (int) m_seqFrag << std::dec << ", SeqNumber=" << m_seqSeq;
+ break;
case WIFI_MAC_DATA:
PrintFrameControl (os);
os << " Duration/ID=" << m_duration << "us";