Wireshark-dev: [Wireshark-dev] Annoying deviances from the 802.11-2016 spec in the names of cip

From: Richard Sharpe <realrichardsharpe@xxxxxxxxx>
Date: Thu, 29 Nov 2018 07:48:40 -0800
Hi folks,

I notice that the names used in the code for the 802.11 dissector
deviate in annoying ways from IEEE 802.11-2016.

For example:

static const value_string ieee80211_rsn_cipher_vals[] = {
  {0, "NONE"},     // Spec says: Use group cipher suite
  {1, "WEP (40-bit)"},  // Spec says: WEP-40
  {2, "TKIP"},
  {3, "AES (OCB)"},  // Spec says: Reserved
  {4, "AES (CCM)"},  // Spec says: GCMP-128
  {5, "WEP (104-bit)"}, // Spec says: WEP-104
  {6, "BIP (128)"},   // Spec says: BIP-CMAC-128
  {7, "Group addressed traffic not allowed"},
  {8, "GCMP (128)" }, // Spec says: GCMP-128
  {9, "GCMP (256)" }, // Spec says: GCMP-256
  {10, "CCMP (256)" }, // Spec says: CCMP-256
  {11, "BIP (GMAC-128)" }, // Spec says: BIP-GMAC-128
  {12, "BIP (GMAC-256)" }, // Spec says: BIP-GMAC-256
  {13, "BIP (CMAC-256)" }, // Spec says: BIP-CMAC-256
  {0, NULL}
};

I think we should conform to the spec as far as possible.

Does anyone have any objections to me fixing these?

-- 
Regards,
Richard Sharpe
(何以解憂?唯有杜康。--曹操)(传说杜康是酒的发明者)