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

wifi: QT agent improvements #14263

Merged
merged 11 commits into from
Feb 29, 2024
Merged

Conversation

D-Triveni
Copy link
Contributor

No description provided.

@D-Triveni D-Triveni requested review from a team, tejlmand and carlescufi as code owners February 28, 2024 15:47
@D-Triveni D-Triveni added this to the 2.6.0 milestone Feb 28, 2024
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Feb 28, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 28, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
wfa-qt-control-app nrfconnect/sdk-wi-fiquicktrack-controlappc@b3cea7b nrfconnect/sdk-wi-fiquicktrack-controlappc@d649bd1 (main) nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 28, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-sdk-wifi X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@D-Triveni D-Triveni force-pushed the fix_qt_issues branch 2 times, most recently from 35e5d61 to cc7fdef Compare February 28, 2024 18:51
@krish2718 krish2718 changed the title Fix qt issues wifi: QT agent improvements Feb 29, 2024
modules/wfa-qt/src/indigo_api_callback_dut.c Outdated Show resolved Hide resolved
modules/wfa-qt/src/indigo_api_callback_dut.c Outdated Show resolved Hide resolved
modules/wfa-qt/Kconfig Outdated Show resolved Hide resolved
west.yml Outdated Show resolved Hide resolved
Added Kconfig entry for setting reboot timeout value.

Signed-off-by: Triveni Danda <[email protected]>
Replace sprintf with snprintf to enhance string length
handling.

Signed-off-by: Triveni Danda <[email protected]>
Control socket is opened by supplicant, so sending
request on the control is enough.

Signed-off-by: Triveni Danda <[email protected]>
This to ensure that the copying operation does not exceed
the size of the destination buffer.

Signed-off-by: Triveni Danda <[email protected]>
Use strncpy to prevent buffer overflow vulnerabilities.

Signed-off-by: Triveni Danda <[email protected]>
Replace macro with Kconfig option for setting default
Wi-Fi interface.

Signed-off-by: Triveni Danda <[email protected]>
Implement imperative check for supplicant
readiness prior to QT thread initiation.

Signed-off-by: Triveni Danda <[email protected]>
Use the net API to read mac address instead of device structs.

Signed-off-by: Triveni Danda <[email protected]>
Configure net config timeout to 0 to avoid
waiting for network initialization.

Signed-off-by: Triveni Danda <[email protected]>
@D-Triveni D-Triveni force-pushed the fix_qt_issues branch 2 times, most recently from 03d8f7d to 394f74e Compare February 29, 2024 10:19
This pulls in the change for removing reboot
when sta disconnect is triggered.

Signed-off-by: Triveni Danda <[email protected]>
Increase libc heap size to fix run time association failures.

Signed-off-by: Triveni Danda <[email protected]>
@jfischer-no jfischer-no merged commit 225b837 into nrfconnect:main Feb 29, 2024
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-wfa-qt-control-app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants