wifi_ap_record_t should be zero'ed before use

[TD-er]

Board

Any

Device Description

Hardware Configuration

Version

latest master (checkout manually)

IDE Name

PlatformIO

Operating System

Windows 11

Flash frequency

Any

PSRAM enabled

yes

Upload speed

115200

Description

As a follow-up of #10281 and as discussed with @me-no-dev

the wifi_ap_record_t struct has been extended recently and just to be sure no member is left in an undefined state, we should zero the entire struct just as has been done in PR #10282

Suggested fix, to zero all of _scanResult right after this line:

arduino-esp32/libraries/WiFi/src/WiFiScan.cpp

Line 120 in e70f4d3

WiFiScanClass::_scanResult = new wifi_ap_record_t[WiFiScanClass::_scanCount];

Sketch

-

Debug Message

-

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

• I confirm I have checked existing issues, online documentation and Troubleshooting guide.

[me-no-dev]

this also does not need to zero the structure, because it will be filled by IDF. However, the way that the next if is written, it is possible to leak the structure if IDF fails for some reason. So I will fix that instead :)