Skip to content

Remove Contiki support - #11788

Open
aidangarske wants to merge 2 commits into
wolfSSL:masterfrom
aidangarske:remove/contiki
Open

aidangarske wants to merge 2 commits into
wolfSSL:masterfrom
aidangarske:remove/contiki

Conversation

@aidangarske

Copy link
Copy Markdown
Member
  • Remove the Contiki and uIP socket integration.
  • Remove the associated platform configuration and random seed handling.
  • Make WOLFSSL_CONTIKI and WOLFSSL_UIP produce a clear removal error.
  • Verified with make -j4 check and ./wolfcrypt/test/testwolfcrypt.

@aidangarske aidangarske self-assigned this Oct 16, 2026
@aidangarske
aidangarske marked this pull request as ready for review October 20, 2026 22:08

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@github-actions

Copy link
Copy Markdown

retest this please

@github-actions

github-actions Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

@aidangarske
aidangarske force-pushed the remove/contiki branch 2 times, most recently from 3b49644 to d9d1b63 Compare October 21, 2026 17:03
@aidangarske

Copy link
Copy Markdown
Member Author

Jenkins retest this please

Copilot AI balanced review requested due to automatic review settings November 5, 2026 18:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are reported for the removal of the legacy Contiki/uIP integration.

No open issues 🎉

What changed in this PR

Removes the legacy Contiki/uIP integration from wolfSSL, including socket callbacks and platform-specific configuration. Builds using either removed feature macro now receive an explicit error.

Changes:

  • Remove uIP transport callbacks, context types, and callback registration.
  • Remove Contiki-specific configuration, header exceptions, and random-seed handling.
  • Reject WOLFSSL_CONTIKI and WOLFSSL_UIP, while retaining macro-check recognition for WOLFSSL_UIP.
File Description
wolfssl/​wolfio.h Remove uIP context and callback declarations and Contiki I/O exceptions.
wolfssl/​wolfcrypt/​settings.h Replace Contiki configuration with an explicit removal error.
wolfssl/​ssl.h Remove the Contiki exception for including sys/uio.h.
wolfcrypt/​src/​random.c Remove Contiki from platform-specific seed handling.
src/​wolfio.c Remove uIP transport and cookie callbacks and the Contiki header exception.
src/​internal.c Remove automatic uIP callback registration.
.wolfssl_known_macro_extras Add WOLFSSL_UIP to the known-macro list.

Review effort: Balanced

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

Successfully merging this pull request may close these issues.

3 participants