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

Mapped lwip buffer error to nsapi no memory error #5033

Merged
merged 1 commit into from Sep 29, 2017

Conversation

Projects
None yet
5 participants
@mikaleppanen
Contributor

mikaleppanen commented Sep 6, 2017

Description

Mapped lwip internal ERR_BUF error value to NSAPI_ERROR_NO_MEMORY.

This corrects issue: #4981

Status

READY

Migrations

NO

Related PRs

None

Todos

  • Tests
  • Documentation

Deploy notes

None

Steps to test or reproduce

None

@kjbracey-arm

I am left wondering what the intended distinction between ERR_BUF and ERR_MEM was - searching for both in the lwIP source didn't really help.

@0xc0170 0xc0170 added needs: CI and removed needs: review labels Sep 19, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 28, 2017

/morph test-nightly

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 28, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1426

All builds and test passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Sep 29, 2017

@theotherjimmy theotherjimmy merged commit c3dc45b into ARMmbed:master Sep 29, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test-nightly Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment