Skip to content

net/gcoap: increase stack size by sizeof(coap_pkt_t) [BACKPORT]#9000

Merged
cgundogan merged 1 commit intoRIOT-OS:2018.04-branchfrom
kaspar030:backport_gcoap_fix
Apr 23, 2018
Merged

net/gcoap: increase stack size by sizeof(coap_pkt_t) [BACKPORT]#9000
cgundogan merged 1 commit intoRIOT-OS:2018.04-branchfrom
kaspar030:backport_gcoap_fix

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

Contribution description

(Backport of #8998.)

Apparently the latest nanocoap rework had side effects on gcoap's stack usage, and not of the good kind.

This PR adds sizeof(coap_pkt_t) to the gcoap stacksize. Meant as quickfix for the release, better solution is on its way.
Issues/PRs references

Fixes #8981.

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master labels Apr 23, 2018
@kaspar030 kaspar030 requested review from kb2ma and miri64 April 23, 2018 07:54
@kaspar030 kaspar030 changed the title net/gcoap: increase stack size by sizeof(coap_pkt_t) net/gcoap: increase stack size by sizeof(coap_pkt_t) [BACKPORT] Apr 23, 2018
Copy link
Copy Markdown
Member

@cgundogan cgundogan left a comment

Choose a reason for hiding this comment

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

ACK

@cgundogan cgundogan merged commit 817551a into RIOT-OS:2018.04-branch Apr 23, 2018
@kaspar030 kaspar030 deleted the backport_gcoap_fix branch April 23, 2018 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants