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.

Sign up for GitHub

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

Jump to bottom

File pem.h does not include "confg.h" #1371

Closed
Dmitry-Frantsev opened this issue Feb 14, 2018 · 2 comments · Fixed by #1375
Closed

File pem.h does not include "confg.h" #1371

Dmitry-Frantsev opened this issue Feb 14, 2018 · 2 comments · Fixed by #1375
Labels
bug

Comments

@Dmitry-Frantsev
Copy link

Description

  • Type: Bug
  • Priority: Minor

Bug

File pem.h does not include "confg.h"
It is neccessary to add such code

#if !defined(MBEDTLS_CONFIG_FILE)
#include "config.h"
#else
#include MBEDTLS_CONFIG_FILE
#endif
@RonEld
Copy link
Contributor

RonEld commented Feb 14, 2018

Hi @Dmitry-Frantsev Thank you for reporting this issue!

@ciarmcom
Copy link

ARM Internal Ref: IOTSSL-2096

RonEld pushed a commit to RonEld/mbedtls that referenced this issue Feb 14, 2018
Include configuration file to headers
4e89d9d 
Add inclusion to configration file in header files,
instead of relying on other header files to include
the configuration file. This issue resolves Mbed-TLS#1371
@RonEld RonEld mentioned this issue Feb 14, 2018
Merged
4 tasks
RonEld pushed a commit to RonEld/mbedtls that referenced this issue Jun 26, 2018
Include configuration file to headers
11c6360 
Add inclusion to configration file in header files,
instead of relying on other header files to include
the configuration file. This issue resolves Mbed-TLS#1371
RonEld pushed a commit to RonEld/mbedtls that referenced this issue Jun 26, 2018
Include configuration file to headers
dffaddf 
Add inclusion to configration file in header files,
instead of relying on other header files to include
the configuration file. This issue resolves Mbed-TLS#1371
RonEld pushed a commit to RonEld/mbedtls that referenced this issue Dec 16, 2018
Include configuration file to headers
8b0cf2e 
Add inclusion to configration file in header files,
instead of relying on other header files to include
the configuration file. This issue resolves Mbed-TLS#1371
RonEld pushed a commit to RonEld/mbedtls that referenced this issue Dec 16, 2018
Include configuration file to headers
0559c66 
Add inclusion to configration file in header files,
instead of relying on other header files to include
the configuration file. This issue resolves Mbed-TLS#1371
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Include configuration file to headers RonEld/mbedtls
3 participants
@Dmitry-Frantsev @ciarmcom @RonEld