Skip to content
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

Unable to establish connection when timeout value is too large #11

Closed
anozaki opened this issue Dec 31, 2015 · 3 comments
Closed

Unable to establish connection when timeout value is too large #11

anozaki opened this issue Dec 31, 2015 · 3 comments

Comments

@anozaki
Copy link

anozaki commented Dec 31, 2015

I have been testing on my mac before I attemt to load this on to an iPad. Everything compiles but had connection problem when the timeout value was set (In my case it was 5000).

The problem seems to be in ConnectOrTimeoutOrExitOnError in network_openssl_wrapper.c.
The select function returns EINVAL and according to the mac's man page. "The specified time limit is invalid. One of its components is negative or too large."

had to update timeout code to...
struct timeval timeout = {timeout_ms / 1000, (timeout_ms % 1000) * 1000};

The timeval is created in bunch of different place so I'm guessing this will happen in other places as well.

@johnrotach
Copy link

Hello Akito,

Thank you for raising this issue! This indeed is an error with larger timeouts. We will aim to add this fix to our next release.

BTW, I see you are targeting an iPad application so I thought I'd point our our iOS SDK for AWS IoT which was recently released.

Thank you again.

John
AWS IoT

@anozaki
Copy link
Author

anozaki commented Jan 2, 2016

Thanks for the link. I'll check out the iOS release.

@bhadrip
Copy link
Contributor

bhadrip commented Feb 11, 2016

This bug is fixed with the release of v1.1.0

@bhadrip bhadrip closed this as completed Feb 11, 2016
aggarw13 added a commit to aggarw13/aws-iot-device-sdk-embedded-C that referenced this issue Nov 29, 2019
Rename Foundry SDK APIs and rename library name to "Provisioning" (that was the outcome pf the API review)
lundinc2 added a commit that referenced this issue Oct 19, 2020
lundinc2 added a commit that referenced this issue Oct 19, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 6, 2020
* Added corePKCS11 library as a submodule
* Added PKCS aws#11 system tests
* Added PKCS aws#11 demos
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 6, 2020
* Added corePKCS11 library as a submodule
* Added PKCS aws#11 system tests
* Added PKCS aws#11 demos
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 9, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 10, 2020
* Added corePKCS11 library as a submodule
* Added PKCS aws#11 system tests
* Added PKCS aws#11 demos

Update demos/pkcs11/common/include/core_pkcs11_config.h

Co-authored-by: SarenaAWS <[email protected]>

Update demos/pkcs11/pkcs11_demo_management_and_rng/pkcs11_demo_management_and_rng.c

Co-authored-by: SarenaAWS <[email protected]>

Submodule mbedTLS.

Add PKCS aws#11 OASIS submodule.

Update lexicon.txt
lundinc2 added a commit that referenced this issue Nov 13, 2020
* Feature: CorePKCS

* Added corePKCS11 library as a submodule
* Added PKCS #11 system tests
* Added PKCS #11 demos

Update demos/pkcs11/common/include/core_pkcs11_config.h

Co-authored-by: SarenaAWS <[email protected]>

Update demos/pkcs11/pkcs11_demo_management_and_rng/pkcs11_demo_management_and_rng.c

Co-authored-by: SarenaAWS <[email protected]>

* Update integration-test/pkcs11/test-config/core_pkcs11_config.h

Co-authored-by: Archit Aggarwal <[email protected]>

* Update integration-test/pkcs11/test-config/core_pkcs11_config.h

Co-authored-by: Archit Aggarwal <[email protected]>

* Apply suggestions from code review

Co-authored-by: Archit Aggarwal <[email protected]>

* Add output to guide user to run the objects demo before the sign and verify demo.

* Set result to invalid object handle if unable to find private and public keys.

Co-authored-by: Archit Aggarwal <[email protected]>
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 17, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 17, 2020
lundinc2 added a commit that referenced this issue Nov 19, 2020
* Add PKCS #11 library and demos to doxygen.

* Add graphviz to CI.

* Add plantuml for PKCS demos.

* Update docs/doxygen/demos/pkcs11_demo.dox

Co-authored-by: SarenaAWS <[email protected]>

* Move PKCS #11 submodule forward.

* Update plantuml and update PNGs.

Co-authored-by: SarenaAWS <[email protected]>
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Nov 24, 2020
Update PKCS folder paths.
lundinc2 added a commit that referenced this issue Nov 24, 2020
…1406)

* Move PKCS #11 submodule pointer forward.
* Update PKCS folder paths.
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Dec 10, 2020
lundinc2 added a commit that referenced this issue Dec 11, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Dec 12, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Dec 12, 2020
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Dec 17, 2020
lundinc2 added a commit that referenced this issue Dec 17, 2020
* Fix memory leak in PKCS #11 test.
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Jan 27, 2021
lundinc2 added a commit to lundinc2/aws-iot-device-sdk-embedded-C that referenced this issue Jan 28, 2021
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

No branches or pull requests

3 participants