Commit a40c63f

committed

SocketWrapper MbedClient debugging readSocket

1 parent 2ece915 commit a40c63fCopy full SHA for a40c63f

File tree

1 file changed

+20

-10

lines changed

libraries/SocketWrapper/src
- MbedClient.cpp

1 file changed

+20

-10

lines changed

`‎libraries/SocketWrapper/src/MbedClient.cpp‎`

Lines changed: 20 additions & 10 deletions

Original file line number	Diff line number	Diff line change
`@@ -22,28 +22,30 @@ void arduino::MbedClient::readSocket() {`
`22`	`22`	`int ret = NSAPI_ERROR_WOULD_BLOCK;`
`23`	`23`	`do {`
`24`	`24`	`mutex->lock();`
`25`		`- if (sock != nullptr && rxBuffer.availableForStore() == 0) {`
	`25`	`+ if (sock == nullptr) {`
	`26`	`+ mutex->unlock();`
	`27`	`+ goto cleanup;`
	`28`	`+ }`
	`29`	`+ if (rxBuffer.availableForStore() == 0) {`
`26`	`30`	`mutex->unlock();`
`27`	`31`	`yield();`
`28`	`32`	`continue;`
`29`		`- } else if (sock == nullptr) {`
`30`		`- goto cleanup;`
`31`	`33`	`}`
`32`	`34`	`ret = sock->recv(data, rxBuffer.availableForStore());`
`33`	`35`	`if (ret < 0 && ret != NSAPI_ERROR_WOULD_BLOCK) {`
	`36`	`+ mutex->unlock();`
`34`	`37`	`goto cleanup;`
`35`	`38`	`}`
`36`	`39`	`if (ret == NSAPI_ERROR_WOULD_BLOCK \|\| ret == 0) {`
`37`		`- yield();`
`38`	`40`	`mutex->unlock();`
`39`		`- continue;`
	`41`	`+ break;`
`40`	`42`	`}`
`41`	`43`	`for (int i = 0; i < ret; i++) {`
`42`	`44`	`rxBuffer.store_char(data[i]);`
`43`	`45`	`}`
`44`	`46`	`mutex->unlock();`
`45`	`47`	`_status = true;`
`46`		`- } while (ret == NSAPI_ERROR_WOULD_BLOCK \|\| ret > 0);`
	`48`	`+ } while (true);`
`47`	`49`	`}`
`48`	`50`	`cleanup:`
`49`	`51`	`_status = false;`
`@@ -98,6 +100,7 @@ int arduino::MbedClient::connect(SocketAddress socketAddress) {`
`98`	`100`	`}`
`99`	`101`
`100`	`102`	`if (static_cast<TCPSocket *>(sock)->open(getNetwork()) != NSAPI_ERROR_OK) {`
	`103`	`+ _status = false;`
`101`	`104`	`return 0;`
`102`	`105`	`}`
`103`	`106`
`@@ -117,6 +120,7 @@ int arduino::MbedClient::connect(SocketAddress socketAddress) {`
`117`	`120`	`configureSocket(sock);`
`118`	`121`	`_status = true;`
`119`	`122`	`} else {`
	`123`	`+ sock->close();`
`120`	`124`	`_status = false;`
`121`	`125`	`}`
`122`	`126`
`@@ -148,6 +152,7 @@ int arduino::MbedClient::connectSSL(SocketAddress socketAddress) {`
`148`	`152`	`}`
`149`	`153`
`150`	`154`	`if (static_cast<TLSSocket *>(sock)->open(getNetwork()) != NSAPI_ERROR_OK) {`
	`155`	`+ _status = false;`
`151`	`156`	`return 0;`
`152`	`157`	`}`
`153`	`158`
`@@ -179,6 +184,7 @@ int arduino::MbedClient::connectSSL(SocketAddress socketAddress) {`
`179`	`184`	`configureSocket(sock);`
`180`	`185`	`_status = true;`
`181`	`186`	`} else {`
	`187`	`+ sock->close();`
`182`	`188`	`_status = false;`
`183`	`189`	`}`
`184`	`190`
`@@ -209,12 +215,14 @@ size_t arduino::MbedClient::write(uint8_t c) {`
`209`	`215`	`}`
`210`	`216`
`211`	`217`	`size_t arduino::MbedClient::write(const uint8_t *buf, size_t size) {`
`212`		`- if (sock == nullptr)`
	`218`	`+ if (mutex == nullptr) {`
`213`	`219`	`return 0;`
`214`		`-`
	`220`	`+ }`
	`221`	`+ mutex->lock();`
`215`	`222`	`sock->set_timeout(_timeout);`
`216`	`223`	`int ret = sock->send(buf, size);`
`217`	`224`	`sock->set_blocking(false);`
	`225`	`+ mutex->unlock();`
`218`	`226`	`return ret >= 0 ? ret : 0;`
`219`	`227`	`}`
`220`	`228`
`@@ -224,8 +232,9 @@ int arduino::MbedClient::available() {`
`224`	`232`	`}`
`225`	`233`
`226`	`234`	`int arduino::MbedClient::read() {`
`227`		`- if (sock == nullptr)`
	`235`	`+ if (mutex == nullptr) {`
`228`	`236`	`return -1;`
	`237`	`+ }`
`229`	`238`	`mutex->lock();`
`230`	`239`	`if (!available()) {`
`231`	`240`	`mutex->unlock();`
`@@ -238,8 +247,9 @@ int arduino::MbedClient::read() {`
`238`	`247`	`}`
`239`	`248`
`240`	`249`	`int arduino::MbedClient::read(uint8_t *data, size_t len) {`
`241`		`- if (sock == nullptr)`
	`250`	`+ if (mutex == nullptr) {`
`242`	`251`	`return 0;`
	`252`	`+ }`
`243`	`253`	`mutex->lock();`
`244`	`254`	`int avail = available();`
`245`	`255`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit a40c63f

File tree

1 file changed

1 file changed

`‎libraries/SocketWrapper/src/MbedClient.cpp‎`

0 commit comments