Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple AP with the same SSID and WPA2/3 mixmode returns undesired alerts. #642

Closed
RobertWi opened this issue Feb 6, 2022 · 11 comments
Closed
Milestone

Comments

@RobertWi
Copy link

RobertWi commented Feb 6, 2022

Observation.

Clients are able to choose WPA3 or WPA2 (mixed mode) on 5.5 SSID but this results in many of these alerts
CRYPTO_CHANGE_BEACON
Using same SSID with different mac results in many
UNEXPECTED_SSID_BEACON
using different mac on same SSID results in many
UNEXPECTED_BSSID_BEACON

AP1
radio 1 SSID Network23 2.4 WPA2-PSK-CCMP c4:41:1e:f8:9b:9c
radio 1 SSID Network23_W 2.4 WPA2-PSK-CCMP c4:41:1e:f8:9b:9c
radio 2 SSID Network23 5.5 WPA2-PSK-CCMP and WPA3-PSK-PSKSHA256-SAE-CCMP c4:41:1e:f8:9b:9d
AP2
radio 1 SSID Network23 2.4 WPA2-PSK-CCMP c4:41:1e:f5:36:44
radio 1 SSID Network23_W 2.4 WPA2-PSK-CCMP c4:41:1e:f5:36:44
radio 2 SSID Network23 5.5 WPA2-PSK-CCMP and WPA3-PSK-PSKSHA256-SAE-CCMP 2 C4:41:1E:F5:36:45

UNEXPECTED_BSSID_BEACON
SSID [Network23_W] was advertised with beacon frame by unexpected BSSID [c6:41:1e:f8:9b:9c]
true can also be c4:41:1e:f5:36:44

CRYPTO_CHANGE_BEACON
SSID [Network23] was advertised with unexpected security settings [WPA2-PSK-CCMP]
SSID [Network23] was advertised with unexpected security settings [WPA3-PSK-PSKSHA256-SAE-CCMP]
true can be both on 2.4 and 5.5 with the same SSID

Most interesting bits from nzyme.conf

  {
    ssid: Network23
    channels: [1,2,3,4,5,6,7,8,9,10,11,12,13,36,40,44,48,52,56,60,64,100,104,108,112,116,120,124,128,132,136,140]
    security: [
               WPA2-PSK-CCMP 
               WPA3-AES-CCMP
              ]
    beacon_rate: 40
    bssids: [
      {
        address: "c4:41:1e:f5:36:44" 
        fingerprints: [ 
          4a60082d10b05d7b8714bcbc778729ca91df196fba81e2eda2ac6ff62b4e3f19
          ]
      }       
      {
        address: "c4:41:1e:f5:36:45" 
        fingerprints: [ 
          775ebfd7baea1ac1d72dca2e6b71c28f91ed12893ef41ebf8babfa4a2809ab92
          ]
      }       
      {
        address: "c4:41:1e:f8:9b:9c"
        fingerprints: [ 
          4a60082d10b05d7b8714bcbc778729ca91df196fba81e2eda2ac6ff62b4e3f19 
          ]
      }
      {
        address: "c4:41:1e:f8:9b:9d"
        fingerprints: [ 
          775ebfd7baea1ac1d72dca2e6b71c28f91ed12893ef41ebf8babfa4a2809ab92
          ]
      }
    ]
  }
  {
    ssid: Network23_W
    channels: [1,2,3,4,5,6,7,8,9,10,11,12,13]
    security: [WPA2-PSK-CCMP]
    beacon_rate: 40
    bssids: [
      {
        address: "c6:41:1e:f5:36:44"
        fingerprints: [ 
          4a60082d10b05d7b8714bcbc778729ca91df196fba81e2eda2ac6ff62b4e3f19 
          ]
      }
      {
        address: "c4:41:1e:f8:9b:9c"
        fingerprints: [ 
          4a60082d10b05d7b8714bcbc778729ca91df196fba81e2eda2ac6ff62b4e3f19
          ]
      }
    ]
  }
]

in ui the sec settings a presented like this.

network23_WPA3

@lennartkoopmann lennartkoopmann added this to the 1.3.0 milestone Feb 6, 2022
@lennartkoopmann
Copy link
Member

It looks like nzyme is getting confused here with two different SSIDs served by the same BSSID. I'm looking into it.

@lennartkoopmann
Copy link
Member

Took a deeper look and actually nzyme appears to operate correctly here. Let's look at the two alerts you posted:

SSID [Network23_W] was advertised with beacon frame by unexpected BSSID [c6:41:1e:f8:9b:9c]

This is correct. Your configuration for Network23_W only expects c6:41:1e:f5:36:44 and c4:41:1e:f8:9b:9c. Add c6:41:1e:f8:9b:9c to the list.

SSID [Network23] was advertised with unexpected security settings [WPA2-PSK-CCMP]
SSID [Network23] was advertised with unexpected security settings [WPA3-PSK-PSKSHA256-SAE-CCMP]

This is correct. Your configuration for Network23 only expects WPA2-PSK-CCMP and WPA3-AES-CCMP.

You should be able to simply extend your configuration and no longer receive the alerts.

@RobertWi
Copy link
Author

RobertWi commented Feb 6, 2022

Took a deeper look and actually nzyme appears to operate correctly here. Let's look at the two alerts you posted:

Thanks

SSID [Network23_W] was advertised with beacon frame by unexpected BSSID [c6:41:1e:f8:9b:9c]

This is correct. Your configuration for Network23_W only expects c6:41:1e:f5:36:44 and c4:41:1e:f8:9b:9c. Add c6:41:1e:f8:9b:9c to the list.

Think this is already in config see Network23_W entry where both c6:41:1e:f5:36:44 and c4:41:1e:f8:9b:9c are listed.

SSID [Network23] was advertised with unexpected security settings [WPA2-PSK-CCMP]
SSID [Network23] was advertised with unexpected security settings [WPA3-PSK-PSKSHA256-SAE-CCMP]

This is correct. Your configuration for Network23 only expects WPA2-PSK-CCMP and WPA3-AES-CCMP.

You should be able to simply extend your configuration and no longer receive the alerts.

Ok, replaced WPA3-AES-CCMP to WPA3-PSK-PSKSHA256-SAE-CCMP is this what you intended. Extending with multiple SSID with the same name is not possible.

@lennartkoopmann
Copy link
Member

lennartkoopmann commented Feb 6, 2022

Think this is already in config see Network23_W entry where both c6:41:1e:f5:36:44 and c4:41:1e:f8:9b:9c are listed.

Those two are, but the alerted BSSID c6:41:1e:f8:9b:9c is not. Note the difference between c6 and c4 at the beginning of the BSSID address.

Ok, replaced WPA3-AES-CCMP to WPA3-PSK-PSKSHA256-SAE-CCMP is this what you intended. Extending with multiple SSID with the same name is not possible.

Not sure if I understand correctly. Did this solve the false alerts?

@peter-reimann-1
Copy link

peter-reimann-1 commented Feb 7, 2022

Hi here my screenshots. I think it is the same problem. Two different security settings within one SSID but two BSSID.

Bildschirmfoto 2022-02-07 um 07 05 58

SSID [Router1] was advertised with unexpected security settings [WPA2-PSK-CCMP].
SSID [Router1] was advertised with unexpected security settings [WPA3-PSK-SAE-CCMP].
Alert Type ID CRYPTO_CHANGE_BEACON

Bildschirmfoto 2022-02-07 um 07 09 29

Bildschirmfoto 2022-02-07 um 07 10 01

SSID [Router1] was advertised with unexpected security settings [WPA2-PSK-CCMP].
SSID [Router1] was advertised with unexpected security settings [WPA3-PSK-SAE-CCMP].
Alert Type ID CRYPTO_CHANGE_PROBERESP

Bildschirmfoto 2022-02-07 um 07 12 50

The related config:

802_11_networks: [
  {
    ssid: Router1
    channels: [1,2,3,4,5,6,7,8,9,10,11,12,13]
    security: [WPA3-PSK-SAE-CCMP,WPA2-PSK-CCMP]
    beacon_rate: 60
    bssids: [
      {
        address: "74:42:7f:03:ae:52",
        fingerprints: [ ...e3b ]
      }
      {
        address: "e8:df:70:c6:ac:f6",
        fingerprints: [ ...b3f ]
      }
    ]
  }
...

I think the array of the security mechanism isn't handled in the right way. For me it seems that the alarms are coming alternating. So if WPA3 is expected WPA2 is seen -> Alarm!. But WPA2 is stored internally and if WPA3 comes up the alarm raises again and store WPA3 for the next comparison.

@RobertWi
Copy link
Author

RobertWi commented Feb 7, 2022

Firstly error-ed indeed on c6 or c4 in mac address config caused by some reworked SSIDs with alternative mac address set.
Secondly mixed mode an AP allowed to use WPA3-PSK-PSKSHA256-SAE-CCMP and WPA3-AES-CCMP so fixed setting now now to WPA3-AES-CCMP only
With these out of the way I think I concur with peter.
Config allows to set different security setting in the array on a single SSID but if it's one or the other in the array these alerts are seen
CRYPTO_CHANGE_BEACON
CRYPTO_CHANGE_PROBERESP

@RobertWi
Copy link
Author

RobertWi commented Feb 7, 2022

Duplicate #527

@lennartkoopmann
Copy link
Member

I think I understand the problem now. Could one of you send me a PCAP of this environment so I can confirm?

It appears like the networks are advertised with different frames per security mechanism. Nzyme expects them to be advertised all in one frame. A PCAP would let me confirm this.

@StarkZarn
Copy link

Any traction on this? I'm still seeing this with a fresh install as of yesterday.

@tzepterbvv
Copy link

Any traction on this? I'm still seeing this with a fresh install as of yesterday.

Nothing heard about it, unfortunally.

@lennartkoopmann
Copy link
Member

This will be solved with the architecture in v2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants