Skip to content

eswifi misc fixes #26712

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

Merged
merged 4 commits into from
Jul 30, 2020
Merged

eswifi misc fixes #26712

merged 4 commits into from
Jul 30, 2020

Conversation

loicpoulain
Copy link
Collaborator

@loicpoulain loicpoulain commented Jul 7, 2020

  • Clear TCP/UDP socket connection state before connection
  • Use log_strdup for strings
  • Add a debug shell to interactively send at command to eswifi controller
  • Fix possible buffer overflow in ipv4/ssid parsing helpers

Loic Poulain added 2 commits July 7, 2020 17:24
Stop any TCP/UDP client session before starting a new one.

Signed-off-by: Loic Poulain <[email protected]>
Use log_strdup() helper for logger string parameters.

Signed-off-by: Loic Poulain <[email protected]>
@nashif nashif added the DNM This PR should not be merged (Do Not Merge) label Jul 7, 2020
@loicpoulain loicpoulain removed the DNM This PR should not be merged (Do Not Merge) label Jul 8, 2020
@galak galak added the DNM This PR should not be merged (Do Not Merge) label Jul 8, 2020
@loicpoulain loicpoulain changed the title [DONOTMERGE ]Wifi eswifi eswifi misc fixes Jul 21, 2020
@loicpoulain loicpoulain removed the DNM This PR should not be merged (Do Not Merge) label Jul 21, 2020
@loicpoulain
Copy link
Collaborator Author

Removing DNM, expecting the PR to me merged now.

@loicpoulain loicpoulain requested a review from pfalcon July 21, 2020 12:40
@loicpoulain loicpoulain added the area: Wi-Fi Wi-Fi label Jul 21, 2020
@loicpoulain loicpoulain requested review from tbursztyka and nashif July 21, 2020 12:41
Copy link
Member

@jukkar jukkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nit, LGTM

@@ -22,4 +22,11 @@ config WIFI_ESWIFI_THREAD_PRIO
This option sets the priority of the esWiFi threads.
Do not touch it unless you know what you are doing.

config WIFI_ESWIFI_SHELL
bool "esWiFi shell"
default n
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Value is "n" by default, this line can be omitted

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, fixed now

Loic Poulain added 2 commits July 27, 2020 19:20
Add a esWiFi shell for device specific controls.
For now used to send at commands.

Signed-off-by: Loic Poulain <[email protected]>
There are possible buffer overflows when parsing the ip address and
SSID. Ensure that we never overwrite the ip and SSID buffers.

Signed-off-by: Loic Poulain <[email protected]>
@loicpoulain loicpoulain requested a review from jukkar July 30, 2020 13:01
@jukkar jukkar merged commit 40e2d94 into zephyrproject-rtos:master Jul 30, 2020
@zephyrbot
Copy link
Collaborator

The backport to v1.14-branch failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-v1.14-branch v1.14-branch
# Navigate to the new working tree
cd .worktrees/backport-v1.14-branch
# Create a new branch
git switch --create backport-26712-to-v1.14-branch
# Cherry-pick the merged commits of this pull request and resolve the conflicts
git cherry-pick 40e2d94bc88ac17a3e96bb95d08f1a21c9c6976f~4..40e2d94bc88ac17a3e96bb95d08f1a21c9c6976f
# Push it to GitHub
git push --set-upstream origin backport-26712-to-v1.14-branch
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-v1.14-branch

Then, create a pull request where the base branch is v1.14-branch and the compare/head branch is backport-26712-to-v1.14-branch.

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

Successfully merging this pull request may close these issues.

7 participants