localhorst/smart-oil-heating-control-system
SHA256
1
0
Fork 0
Code Issues 4 Pull Requests 5 Actions Packages Projects Releases 1 Wiki Activity

Error handling on system boot #30

Open
localhorst wants to merge 21 commits from feature/error-handling into main
pull from: feature/error-handling
merge into: localhorst:main
Conversation 1 Commits 21 Files Changed 17 +1608 -570
localhorst commented 2026-01-10 13:33:50 +01:00
Owner
Copy Link

Implements #3

Implements https://git.mosad.xyz/localhorst/smart-oil-heating-control-system/issues/3
localhorst added the
enhancement
 label 2026-01-10 13:33:50 +01:00
localhorst self-assigned this 2026-01-10 13:33:50 +01:00
localhorst added 19 commits 2026-01-10 13:33:51 +01:00
Wrong memset size cd73985740
Off-by-one error (buffer overread) 0775fda0ca
Misuse of ESP_ERROR_CHECK 09a3c3a22d
ncorrect memset with strlen 781f9a1445
Missing mutex protection 267197ec20
Unprotected shared state access 8c3dbc2886
Non-thread-safe function df3825df3a
Unchecked gpio_config returns 10f9645580
Unchecked xEventGroupCreate 67929580d5
Unchecked network configuration 020eb63e05
Unchecked WiFi API call 05757a5038
Unsafe strcpy 0236ebcdd1
Floating-point equality comparison a9ec101bc6
uUnchangedCounter reset logic flaw 40f757b7d1
Variable name shadows type name d36b91a0fd
Missing break before default b718073907
Potential division by zero f3f6f1bc5f
implement config f8f6af53bd
error handling and cleanup 1d4e272d80
localhorst commented 2026-01-10 13:34:44 +01:00
Author
Owner
Copy Link

Testing needed! See Bug

Testing needed! See Bug
localhorst added 1 commit 2026-01-10 13:42:28 +01:00
cleanup README 430b4cb690
localhorst added 1 commit 2026-01-10 18:51:37 +01:00
update README 4f355bdfdf
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/error-handling:feature/error-handling
git checkout feature/error-handling
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
localhorst
No Assignees localhorst
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: localhorst/smart-oil-heating-control-system#30
No description provided.
Delete Branch "feature/error-handling"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?